-
Notifications
You must be signed in to change notification settings - Fork 39
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
Build fails with llvm-17 #115
Comments
llvm-17.0.6 has already been stabilized in gentoo. Any updates possible in the next few weeks? |
Yeah sorry for the delay; looks like arch has llvm-17 finally so it should be reasonably straightforward this weekend. Also apparently I entirely misread this before: the only actual error appears to be the It's probable this would fix it: clang::Expr* e = clang::cast<clang::Expr*>(stmt->Value);
// ^ notice If it does (and everything else works!) and you want to submit a PR I'd merge it too.. though fixing the deprecated header would be nice as well. |
Thanks, while I would have been able to fix that line, I was thrown off by some weird linker error which resulted from simply not adapting the LLVM version string in the CMakeLists.txt. My 3 commits in the PR should fix the issues you mentioned, it built fine here. |
This should build/work now. |
As described in #114, here is the output (there are also some deprecated header warnings in addition to the compile error in AST.cpp):
The text was updated successfully, but these errors were encountered: