-
Notifications
You must be signed in to change notification settings - Fork 92
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
C++14 Workaround issue #349 #357
Conversation
@@ -46,6 +46,14 @@ | |||
|
|||
#include<Kokkos_Core.hpp> | |||
|
|||
#ifdef KOKKOS_ENABLE_CXX14 |
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 is not a blocker, but one-liners are nice:
#if defined(KOKKOS_ENABLE_CXX14) && defined(KOKKOS_COMPILER_GNU) && KOKKOS_COMPILER_GNU <= 720
# define KOKKOS_IMPL_BATCHED_GEMM_GCC_CXX14_WORKAROUND
#endif
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 and I will test on white.
@kyungjoo-kim Were you able to test this on white. It would be nice to merge this in. |
Kokkos::parallel_for(Kokkos::ThreadVectorRange(team,blockDim_i), [&] (const int i) { | ||
#ifdef KOKKOS_IMPL_BATCHED_GEMM_GCC_CXX14_WORKAROUND | ||
const int idx_j = offset_j+j; | ||
#endif |
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.
It probably necessary to pu int idx_j = offset_j + j
with #else
.
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.
Oh... sorry... I just saw that the idx_j
is defined on the above.
It is okay with white. Plz, merge this PR. |
@kyungjoo-kim Thank you ! |
Fixes C++14 build issues.