-
Notifications
You must be signed in to change notification settings - Fork 407
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
Desul atomics: let relocatable device code mode be part of the configuration #5991
Desul atomics: let relocatable device code mode be part of the configuration #5991
Conversation
…ration is not compatible with the relocatable device code mode
…E_DEVICE_CODE macro guards
…ndled version of the desul atomics
c302dd5
to
22b8c70
Compare
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.
This file was removed in desul/desul#98
What release changelog should this be on? 4.1? |
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.
Looks good to me. Most of the changes are cherry-picked from upstream desul anyway.
If we want to mention it then yes. |
Ignoring GCC-8.4.0 failure |
…uration (kokkos#5991) * Desul atomics: Make compilation fail if SEPARABLE_COMPILATION configuration is not compatible with the relocatable device code mode * Desul atomics: Prefer #ifdef DESUL_ATOMICS_ENABLE_{CUDA,HIP}_SEPARABLE_DEVICE_CODE macro guards * Desul atomics: add DESUL_ATOMICS_ENABLE_{CUDA,HIP}_SEPARABLE_COMPILATION macros * Configure DESUL_ATOMICS_ENABLE_{CUDA,HIP}_SEPARABLE_COMPILATION in bundled version of the desul atomics * Merge latest version * Fixup Clang+CUDA defines __CLANG_RDC__ instead of __CUDACC_RDC__
Catch incompatible RDC mode at compile time for CUDA and HIP