-
Notifications
You must be signed in to change notification settings - Fork 136
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
Try and fix CI for old CTK #116
Conversation
c6968ab
to
d267ed6
Compare
6c9a3d3
to
fbbaf6b
Compare
libcudacxx/include/cuda/std/detail/libcxx/include/__concepts/convertible_to.h
Show resolved
Hide resolved
60b2efe
to
c2e730f
Compare
libcudacxx/.upstream-tests/test/std/atomics/atomics.lockfree/isalwayslockfree.pass.cpp
Outdated
Show resolved
Hide resolved
libcudacxx/include/cuda/std/detail/libcxx/include/__expected/expected_base.h
Show resolved
Hide resolved
libcudacxx/include/cuda/std/detail/libcxx/include/__type_traits/is_convertible.h
Show resolved
Hide resolved
libcudacxx/include/cuda/std/detail/libcxx/include/__type_traits/is_swappable.h
Show resolved
Hide resolved
52a7754
to
1709b83
Compare
For the record, I did additional testing with nvrtc and nvcc, which passed with nvcc 12.1 and all standard modes |
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.
I haven't delved into all of the details of the changes here. I'll leave that to @wmaxey @griwes @senior-zero
To summarize the warning suppression discussion We can suppress the warnings with nvcc 11.3 onwards. Otherwise it is not supported. That only leaves 11.1 and 11.2 as problematic versions. My agreement with @senior-zero was that those two versions are fine to be "potentially broken" especially given that it is really easy to work around the issue on the user side |
Older nvcc does not know `nv_diag_supppress` but only `diag_suppress` so special case it
Old nvcc does not know constexpr variables on device, so move to `const` in those cases
Move to a member function
In particular, constrained friend functions are critical
The constrain breaks in earlier standard modes, so as a workaround remove the constraint from our internal `__construct_at` prior to c++20 and switch back to the proper `construct_at` with c++20 onwards
Adopt histogram script to new format
No description provided.