Skip to content

Commit 5735f8c

Browse files
RazishSomaZ
authored andcommitted
split UseSanitizer into UseAddressSanitizer and UseUndefinedSanitizer
1 parent 4b8d64d commit 5735f8c

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

CMakeLists.txt

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,8 @@ option(BuildJK2SPGame "Whether to create projects for the jk2 sp gamecode mod (j
5050
option(BuildJK2SPRdVanilla "Whether to create projects for the jk2 sp renderer (rdjosp-vanilla_x86.dll)" OFF)
5151

5252
option(BuildTests "Whether to build automatic unit tests (requires Boost)" OFF)
53-
option(UseSanitizer "Whether to enable runtime sanitizers (e.g. AddressSanitizer)" OFF)
53+
option(UseAddressSanitizer "Whether to enable runtime address sanitizer" OFF)
54+
option(UseUndefinedSanitizer "Whether to enable runtime Undefined Behavior sanitizer" OFF)
5455

5556
include(CMakeDependentOption)
5657
cmake_dependent_option(BuildSymbolServer "Build WIP Windows Symbol Server (experimental and unused)" OFF "NOT WIN32 OR NOT MSVC" OFF)
@@ -245,9 +246,14 @@ elseif (("${CMAKE_C_COMPILER_ID}" STREQUAL "GNU") OR ("${CMAKE_C_COMPILER_ID}" M
245246
set(CMAKE_SHARED_LIBRARY_LINK_C_FLAGS "")
246247
set(CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS "")
247248

248-
if(CMAKE_BUILD_TYPE MATCHES "DEBUG" OR CMAKE_BUILD_TYPE MATCHES "Debug" AND UseSanitizer)
249+
if(UseAddressSanitizer)
250+
# also raise stack size drastically (to 64MiB), since the sanitizer adds overhead to stack frames
249251
add_compile_options(-fsanitize=address)
250-
add_link_options(-fsanitize=address)
252+
add_link_options(-fsanitize=address -z stack-size=4000000)
253+
endif()
254+
if(UseUndefinedSanitizer)
255+
add_compile_options(-fsanitize=undefined)
256+
add_link_options(-fsanitize=undefined)
251257
endif()
252258

253259
# additional flags for debug configuration

0 commit comments

Comments
 (0)