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

Fix discrepancy in EmitC parser/printer #190

Merged
merged 3 commits into from
May 22, 2024

Conversation

cferry-AMD
Copy link
Collaborator

emitc.size_t is the default type in emitc::ForLoop induction variables now, so the type is not printed in the MLIR "assembly". However, index type is still assumed at parse time barring further type information in the IR. This commit reconciles the print and parse behavior to assume emitc.size_t as the default type.

@cferry-AMD cferry-AMD requested a review from josel-amd May 22, 2024 06:53
@cferry-AMD cferry-AMD requested a review from mgehre-amd May 22, 2024 09:56
@mgehre-amd
Copy link
Collaborator

Changing the meaning of of current IR by changing the default type might be controversial. Fine for me, but something to explicitly mention when upstreaming.

@cferry-AMD
Copy link
Collaborator Author

Changing the meaning of of current IR by changing the default type might be controversial. Fine for me, but something to explicitly mention when upstreaming.

We can defer that change to the PR that drops index type support when upstreaming. Let's make a specific discussion on the Discord about that.

@cferry-AMD cferry-AMD merged commit 17f7259 into feature/fused-ops May 22, 2024
3 checks passed
@cferry-AMD cferry-AMD deleted the corentin.fix_emitc_discrepancy branch May 22, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants