Turn off fast math for Catch2 build

alpaka-group · Sep 5, 2023 · 8bcee69 · 8bcee69
1 parent 1f332be
commit 8bcee69
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/test/common/devCompileOptions.cmake b/test/common/devCompileOptions.cmake
@@ -146,7 +146,7 @@ if(CMAKE_CXX_COMPILER_ID MATCHES "Clang" OR CMAKE_CXX_COMPILER_ID STREQUAL "Inte
 
     if(${CMAKE_CXX_COMPILER_ID} STREQUAL "IntelLLVM")
         # fast math is turned on by default with ICPX, which breaks our unit tests
-        list(APPEND alpaka_DEV_COMPILE_OPTIONS "-ffp-model=precise")
+        list(APPEND alpaka_DEV_COMPILE_OPTIONS "-fp-model=precise")
 
         if (alpaka_ACC_SYCL_ENABLE)
             # avoid: warning: disabled expansion of recursive macro

diff --git a/thirdParty/CMakeLists.txt b/thirdParty/CMakeLists.txt
@@ -26,6 +26,9 @@ if(BUILD_TESTING)
         # Remarks are not treated as warnings so we need to disable them manually. 
         add_compile_options($<$<AND:$<CONFIG:Debug>,$<CXX_COMPILER_ID:IntelLLVM>>:-Rno-debug-disables-optimization>)
 
+        # Turn off fast math to prevent warnings inside Catch2 (icpx defaults to -fp-model=fast)
+        add_compile_options($<$<CXX_COMPILER_ID:IntelLLVM>:-fp-model=precise>)
+
         add_subdirectory(catch2)
 
         # hide Catch2 cmake variables by default in cmake gui