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
Avoid using __align__
in ROCm
#2638
Conversation
Geez...I wonder if the template |
/test |
Successfully created a job for commit a5c754c: |
@@ -62,7 +62,12 @@ struct _select_greater_type | |||
* | |||
*/ | |||
template <typename T> | |||
#if defined(__CUDACC__) |
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.
Can you also test __CUDACC_RDC__
here? It's a preparation for #2426.
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 think both __CUDACC__
and __CUDACC_RDC__
are set when built with --relocatable-device-code=true
.
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.
Ah, I see! Yes, I can verify. Thanks @kmaehashi! This is not obvious from the documentation...
imosCI, test this please. |
Successfully created a job for commit a5c754c: |
imosCI, test this please. |
Successfully created a job for commit a5c754c: |
pfnCI, test this please. |
Successfully created a job for commit a5c754c: |
Jenkins CI test (for commit a5c754c, target branch master) succeeded! |
Ref #2562