-
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
OpenMPTarget: Block unit tests that do not pass with the nvidia compiler. #3839
Conversation
…ckend with the nvidia compiler.
…enMPTarget backend tests that fail with nvidia compiler in the clang compiler.
…mented for the OpenMPTarget backend.
core/CMakeLists.txt
Outdated
@@ -7,7 +7,11 @@ IF (NOT Kokkos_INSTALL_TESTING) | |||
ENDIF() | |||
|
|||
KOKKOS_ADD_TEST_DIRECTORIES(unit_test) | |||
|
|||
# FIXME_OPENMPTARGET - the nvidia compiler in the OpenMPTarget backend does not pass the perf_tests. |
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.
# FIXME_OPENMPTARGET - the nvidia compiler in the OpenMPTarget backend does not pass the perf_tests. | |
# FIXME_OPENMPTARGET - the PGI compiler in the OpenMPTarget backend does not pass the perf_tests. |
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.
PGI is not a thing anymore
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.
Then let's rename everything to NVHPC.
…tests that do not pass with the nvidia compiler.
…ar the confusion about the PGI/NVHPC compiler.
Retest this please. |
core/unit_test/CMakeLists.txt
Outdated
endif() | ||
|
||
# FIXME_OPENMPTARGET - Comment non-passing tests with the NVIDIA HPC compiler nvc++ | ||
if(KOKKOS_ENABLE_OPENMPTARGET AND KOKKOS_CXX_COMPILER_ID STREQUAL "PGI") |
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.
if(KOKKOS_ENABLE_OPENMPTARGET AND KOKKOS_CXX_COMPILER_ID STREQUAL "PGI") | |
if(KOKKOS_ENABLE_OPENMPTARGET AND | |
(KOKKOS_CXX_COMPILER_ID STREQUAL PGI OR KOKKOS_CXX_COMPILER_ID STREQUAL NVHPC)) |
for CMake 3.20 (https://cmake.org/cmake/help/v3.20/variable/CMAKE_LANG_COMPILER_ID.html#variable:CMAKE__COMPILER_ID).
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.
LGTM.
IF(KOKKOS_ENABLE_OPENMPTARGET | ||
AND NOT KOKKOS_CXX_COMPILER_ID STREQUAL PGI | ||
AND NOT KOKKOS_CXX_COMPILER_ID STREQUAL NVHPC) | ||
KOKKOS_ADD_TEST_DIRECTORIES(perf_test) | ||
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.
Uhhhhh
These had been inadvertently disbaled in kokkos#3839 Co-Authored-By: Nick Curtis <nicholas.curtis@amd.com> Co-Authored-By: Bruno Turcksin <bruno.turcksin@gmail.com>
The PR blocks unit tests that do not pass with the nvidia-compiler based on the compiler-id.