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
Correct _LIBCUDACXX_CUDACC_VER value computation #9579
Correct _LIBCUDACXX_CUDACC_VER value computation #9579
Conversation
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.
🔥 Thanks @robertmaynard
Confirmed that this fixes my build for #9483 |
We should hold off on merging this while upstream decides what approach they want. That way we only have to update the patch file once. |
Happy to review this once libcudacxx decides what the supported way forward will be to check versions. |
Codecov Report
@@ Coverage Diff @@
## branch-21.12 #9579 +/- ##
================================================
- Coverage 10.79% 10.65% -0.14%
================================================
Files 116 117 +1
Lines 18869 19746 +877
================================================
+ Hits 2036 2104 +68
- Misses 16833 17642 +809
Continue to review full report at Codecov.
|
@vyasr @codereport This is ready for review as these changes have been accepted as is upstream ( NVIDIA/libcudacxx#225 ) |
@gpucibot merge |
This brings the libcudacxx patch inline with the proposed upstream changes ( NVIDIA/libcudacxx#225 ).
This is needed so that CUDA versions are correctly computed and decimal-128 support is enabled when using CUDA 11.5+