-
Notifications
You must be signed in to change notification settings - Fork 41
⏪️ [MLIR] Bring back MLIR / LLVM 19.0 support #934
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM. Just some minor fixes necessary.
Probably also worth a changelog entry. |
Co-authored-by: Lukas Burgholzer <[email protected]> Signed-off-by: Patrick Hopf <[email protected]>
Thanks @burgholzer for the quick review. I added the requested changes. |
Sounds good to me! The current code changes all look reasonable. |
We know that |
Apparently, the latest minor version of LLVM 19 does not cause any issues in our tests, and this PR should be ready to go in. Is that correct, @burgholzer? |
Thank you @ystade! Looks like this is only an issue with the v19 that is used (and possibly patched) in Catalyst---the one I have been using on my local machine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍🏼
Description
Reincarnate MLIR / LLVM 19.0 support.
Necessary for Catalyst-related efforts (like #880).
Checklist: