Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[RTG][Elaboration] Add support for 'index.add' and 'index.cmp' #7978

Open
wants to merge 1 commit into
base: maerhart-rtg-elaboration-sequences
Choose a base branch
from

Conversation

maerhart
Copy link
Member

  • Use the index dialect instead of arith because it is a smaller dependency and simpler, but still has everything we need.
  • Add support for index.add and index.cmp
  • Add specialized elaborator values for indices and booleans because creating attributes for intermediates is quite bad performance-wise (ideally we'd have some evaluation interface at some point that we can also use for folding, or better folders that we can call directly)

@maerhart maerhart added the RTG Involving the `rtg` dialect label Dec 12, 2024
@@ -85,7 +86,7 @@ namespace {
/// The abstract base class for elaborated values.
struct ElaboratorValue {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like just a std::variant.

Copy link
Contributor

@darthscsi darthscsi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I think the elaboration is reinventing std::variant, but that is separate from this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RTG Involving the `rtg` dialect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants