-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Build LLVM 22.x 8462cff fix removed/reprecated #17786
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
base: master
Are you sure you want to change the base?
Conversation
|
LLVM needs to be updated on the CI |
|
We're still on LLVM 19, feel free to use ifdef guards with LLVM version to get this merged |
|
@Niram7777 were you able to test this with LLVM22? FreeBSD, Windows Clang and Mac builds are all using Clang 21. I can test the Linux and Windows MSVC builds. Windows MSVC builds fine. Tested LLVM22 1c32b6f. |
d476be4 to
dc72d87
Compare
9223dc5 to
7d82eeb
Compare
| ) | ||
|
|
||
| target_include_directories(soundtouch PRIVATE | ||
| target_include_directories(soundtouch SYSTEM PRIVATE |
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.
The reason why this extra parameter is needed is because we are not using imported targets so the compilation warnings from 3rd parties are propagated.
https://cmake.org/cmake/help/latest/prop_tgt/SYSTEM.html
The PR is ready for merge
getDeclaration
[IR] Remove Intrinsic::getDeclaration llvm/llvm-project#152645
getFirstNonPHI
[NFC][DebugInfo] Deprecate iterator-taking moveBefore and getFirstNonPHI llvm/llvm-project#124290
use pointers intead of LLVM iterator because some of them API dont support their iterator
Fix an issue I had for quite some time:
by configuring 3rdparties as
system, I may have missed others.LLVM used llvm/llvm-project@8462cff