-
Notifications
You must be signed in to change notification settings - Fork 137
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
Make symcc work with llvm 17 #149
Conversation
Thanks for the PR, I added some CI tests for LLVM 16 and 17. |
I made the necessary change here |
this guy is not working eurecom-s3/qsym#1. |
i don't get why this breaks even for llvm 11, 12, 13, 14, 15 where i changed nothing. |
I found out why then i'll update the submodule to symcc |
Indeed, that was the wrong branch, got confused too... Thanks ! |
sry i forgot to add ifdef |
looks good now |
Excellent, thank you! |
llvm/Transforms/IPO/PassManagerBuilder.h doesn't exist on llvm 17