-
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
Add missing specailization for OMPT to Kokkos Random #3967
Add missing specailization for OMPT to Kokkos Random #3967
Conversation
Retest this please. |
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.
Implement the spin loop.
f152499
to
7394dd8
Compare
7394dd8
to
af18499
Compare
// FIXME_OPENMPTARGET: Need this for OpenMPTarget because contra to the standard | ||
// llvm requires the binary operator defined not just the += | ||
KOKKOS_INLINE_FUNCTION | ||
RandomProperties operator+(const RandomProperties& org, |
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.
What is "org" short for?
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.
original I guess?
OpenMPTarget algorithms unit test CMake: less convoluted condition Co-authored-by: Daniel Arndt <arndtd@ornl.gov>
Co-authored-by: Daniel Arndt <arndtd@ornl.gov>
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 OK.
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.
LGTM
algorithms/src/Kokkos_Random.hpp
Outdated
using locks_view_type = View<int*, Kokkos::Experimental::OpenMPTarget>; | ||
KOKKOS_FUNCTION | ||
static int get_state_idx(const locks_view_type& locks) { | ||
const int i = omp_get_team_num() * 1024 + omp_get_thread_num(); |
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.
why 1024 instead of 2048 ?
[3.4.1] Take over #3967: Add missing specailization for OMPT to Kokkos Random
No description provided.