Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Commit

Permalink
Update more CUDA compiler id checks.
Browse files Browse the repository at this point in the history
  • Loading branch information
alliepiper committed Jun 7, 2023
1 parent 7657c7f commit 7d9ad98
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 17 deletions.
8 changes: 0 additions & 8 deletions examples/cmake/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,6 @@ thrust_update_system_found_flags()

set(extra_cmake_flags)

# Need to pass these when testing NVC++.
if ("NVCXX" STREQUAL "${CMAKE_CUDA_COMPILER_ID}")
set(extra_cmake_flags
-D "CMAKE_CUDA_COMPILER_ID=${CMAKE_CUDA_COMPILER_ID}"
-D "CMAKE_CUDA_COMPILER_FORCED=${CMAKE_CUDA_COMPILER_FORCED}"
)
endif()

if (THRUST_CPP_FOUND AND THRUST_CUDA_FOUND)
# Do a basic check of the cmake/ThrustAddSubdir.cmake mechanism:
add_test(
Expand Down
8 changes: 0 additions & 8 deletions testing/cmake/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,6 @@ thrust_update_system_found_flags()

set(extra_cmake_flags)

# Need to pass these when testing NVC++.
if ("NVCXX" STREQUAL "${CMAKE_CUDA_COMPILER_ID}")
set(extra_cmake_flags
-D "CMAKE_CUDA_COMPILER_ID=${CMAKE_CUDA_COMPILER_ID}"
-D "CMAKE_CUDA_COMPILER_FORCED=${CMAKE_CUDA_COMPILER_FORCED}"
)
endif()

if (THRUST_CPP_FOUND AND THRUST_CUDA_FOUND)
# Test that we can use `find_package` on an installed Thrust:
add_test(
Expand Down
2 changes: 1 addition & 1 deletion testing/unittest_static_assert.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@ target_link_libraries(${test_target} PRIVATE thrust.silence_unreachable_code_war

# The machinery behind this test is not compatible with NVC++.
# See https://github.com/NVIDIA/thrust/issues/1397
if ("NVCXX" STREQUAL "${CMAKE_CUDA_COMPILER_ID}")
if ("NVHPC" STREQUAL "${CMAKE_CUDA_COMPILER_ID}")
set_tests_properties(${test_target} PROPERTIES DISABLED True)
endif()

0 comments on commit 7d9ad98

Please sign in to comment.