Enable sanitizers on CI

ImJimmi · Sep 18, 2024 · 1c5b5f4 · 1c5b5f4
1 parent 5a2137a
commit 1c5b5f4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.github/workflows/test-runner.yml b/.github/workflows/test-runner.yml
@@ -23,7 +23,8 @@ jobs:
           - platform: macos-latest
             platform-name: macOS
             package-manager: brew
-            additional-cmake-options: -DJIVE_ENABLE_COVERAGE=ON
+            additional-cmake-options: -DJIVE_ENABLE_COVERAGE=ON -DJIVE_ENABLE_SANITISERS=ON
+
           - platform: windows-latest
             platform-name: Windows
             platform-shell: cmd

diff --git a/cmake/jive_compiler_and_linker_options.cmake b/cmake/jive_compiler_and_linker_options.cmake
@@ -27,13 +27,11 @@ if (APPLE)
 
     if (JIVE_ENABLE_SANITISERS)
         target_compile_options(jive_compiler_and_linker_options
-                               INTERFACE -fsanitize=address
-                                         -fsanitize=undefined
+                               INTERFACE -fsanitize=undefined
         )
 
         target_link_options(jive_compiler_and_linker_options
                             INTERFACE
-                            -fsanitize=address
                             -fsanitize=undefined
         )
     endif ()