cmake: Raise max llvm major version to 16 #2528
Conversation
kraj
requested review from
ajor,
viktormalik,
danobi,
fbs and
BurntBrunch
as code owners
|
Could we add LLVM 16 to the CI? |
Once it releases then yes, it is in RC4 right now. Perhaps final release will happen any time |
Ok, thanks for the info. I'd prefer waiting until that happens. I don't fancy the idea of having support for something that isn't properly tested. |
|
I believe llvm 16 is released now |
|
@kraj please add LLVM 16 to the CI as a part of this PR. |
Sadly, we are running into llvm/llvm-project#61436 which will be fixed in 16.0.1 point release hopefully, since the fix is already attached to the issue. Until then clang-16 loop wont be able to run. |
Ok, hopefully they release it soon. I suggest to wait until then. |
|
@kraj @viktormalik I'd appreciate if you folks could bump that LLVM issue, since there's not a lot of activity from the LLVM folks to help get it merged. |
|
16.0.1 has been out for two weeks but the issue doesn't seem to be fixed (at least restarting the CI didn't help) |
|
We should try our best to have this in the 0.18 release (currently planned for May 1st). |
|
While upstream LLVM should still be fixed, we can work around the bug by doing something like bcc does: |
Sounds good. Let me give it a try so that we can do a release with LLVM 16 support. |
|
Ok, I have it ready and it passes the CI. @kraj do you mind if I push it directly to this PR (which would be your master branch) or should I open a new one? |
please go ahead. |
|
Looks like it's not possible (protected master branch?) so I opened a new PR #2592. Feel free to review, thanks! |
No description provided.