-
Notifications
You must be signed in to change notification settings - Fork 236
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
workflows: test against LLVM 18.1 #525
Conversation
Changes look good to me but tests are not passing |
I was a bit fast reading the changelog for v2.0.0 in |
Maybe we could run the old version for <7 and the new one for newer versions? |
Done in a25308b.
|
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.
Thank you!
Description
Add a matrix entry for LLVM 18.1 in the testing workflow.
Requires updating to KyleMayes/install-llvm-action v2.0.2+. Breaking API changes from v2.0.0 are not relevant here.
Related Issue
#482, #524 (comment)
How This Has Been Tested
install-llvm-action
resolves the LLVM version correctly.cargo test -F llvm18-0
with LLVM 18.1.8 libraries.Checklist