[FEA]: llvm / clang 17 CI coverage #1360
Labels
feature request
New feature or request.
Comments
|
Did they perchance add another builtin that is named like a function? |
|
I have opened a PR to add clang-17 to our CI matrix. |
Thank you! That is a super fast turnaround. I trust that this will be useful, but I am also slightly sorry because in the meantime we have understood the root cause of our troubles. I have summarized that here: You have nicely patched the problem we were seeing a long while ago (for clang 16 support), but rapids memory manager 23.12 vendors an old version of libcudacxx! :) sorry for the noise 🔊 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is this a duplicate?
Area
libcu++
Is your feature request related to a problem? Please describe.
Hello! We see that in CI you are covering the binaries from the LLVM 16.x releases, but there is no coverage yet for the 17.x releases:
cccl/ci/matrix.yaml
Line 34 in c494fe9
That's something that is very interesting for us.
Anecdote: we are debugging a clang-tidy error which is actually a compilation error: clang is tripped up by code in
libcudacxx/cuda/std/detail/libcxx/includeand for our debugging it was interesting to see where you are in terms of supporting clang 17.Describe the solution you'd like
cover clang 17 in your CI
Describe alternatives you've considered
n/a
Additional context
No response
The text was updated successfully, but these errors were encountered: