Update comment around why we disable CMAKE_CUDA_ARCHITECTURES in init

rapidsai · Dec 19, 2022 · 9df2f10 · 9df2f10
1 parent 9bd76e5
commit 9df2f10
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/rapids-cmake/cuda/init_architectures.cmake b/rapids-cmake/cuda/init_architectures.cmake
@@ -84,8 +84,9 @@ function(rapids_cuda_init_architectures project_name)
     set(cuda_arch_mode "RAPIDS")
   endif()
 
-  # This needs to be run before enabling the CUDA language since RAPIDS supports the magic string of
-  # "ALL"
+  # This needs to be run before enabling the CUDA language since RAPIDS supports magic
+  # values like `RAPIDS`, `ALL`, and `NATIVE` which if propagated cause CMake to fail to determine
+  # the CUDA compiler
   if(cuda_arch_mode STREQUAL "RAPIDS")
     set(CMAKE_CUDA_ARCHITECTURES OFF PARENT_SCOPE)
     set(load_file "${CMAKE_CURRENT_FUNCTION_LIST_DIR}/detail/invoke_set_all_architectures.cmake")