-
Notifications
You must be signed in to change notification settings - Fork 12
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
initial LLVM 17 support #274
Conversation
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
@@ -21,7 +21,8 @@ jobs: | |||
run: | | |||
wget https://apt.llvm.org/llvm.sh | |||
chmod +x llvm.sh | |||
sudo ./llvm.sh 15 | |||
sudo ./llvm.sh 17 | |||
sudo apt install -y libpolly-17-dev |
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.
NB: At some point LLVM appears to have dropped Polly from their default library base in llvm.sh
. The llvm-ir
crate needs it, so I've added it manually in each install step.
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.
I have just one small comment.
Please just go ahead an merge once you've had a chance to consider it. |
Signed-off-by: William Woodruff <[email protected]>
Blocked on cdisselkoen/llvm-ir#59.