apply review suggestions

Co-authored-by: Yuhsiang M. Tsai <[email protected]>

cmake/autodetect_executors.cmake

@@ -5,17 +5,23 @@ find_package(OpenMP)
 include(CheckLanguage)
 check_language(CUDA)
 
-if(NOT DEFINED GINKGO_BUILD_OMP AND OpenMP_CXX_FOUND)
- message(STATUS "Enabling OpenMP executor")
+if(OpenMP_CXX_FOUND)
+ if(NOT DEFINED GINKGO_BUILD_OMP)
+ message(STATUS "Enabling OpenMP executor")
+ endif()
  set(GINKGO_HAS_OMP ON)
 endif()
 
-if(NOT DEFINED GINKGO_BUILD_CUDA AND CMAKE_CUDA_COMPILER)
- message(STATUS "Enabling CUDA executor")
+if(CMAKE_CUDA_COMPILER)
+ if(NOT DEFINED GINKGO_BUILD_CUDA)
+ message(STATUS "Enabling CUDA executor")
+ endif()
  set(GINKGO_HAS_CUDA ON)
 endif()
 
-if(NOT DEFINED GINKGO_BUILD_HIP AND GINKGO_HIPCONFIG_PATH)
- message(STATUS "Enabling HIP executor")
+if(GINKGO_HIPCONFIG_PATH)
+ if(NOT DEFINED GINKGO_BUILD_HIP)
+ message(STATUS "Enabling HIP executor")
+ endif()
  set(GINKGO_HAS_HIP ON)
-endif()
+endif()