Skip to content

Commit b5a3093

Browse files
committed
split UseSanitizer into UseAddressSanitizer and UseUndefinedSanitizer
1 parent 2497786 commit b5a3093

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
@@ -49,7 +49,8 @@ option(BuildJK2SPGame "Whether to create projects for the jk2 sp gamecode mod (j
4949
option(BuildJK2SPRdVanilla "Whether to create projects for the jk2 sp renderer (rdjosp-vanilla_x86.dll)" OFF)
5050

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

5455
include(CMakeDependentOption)
5556
cmake_dependent_option(BuildSymbolServer "Build WIP Windows Symbol Server (experimental and unused)" OFF "NOT WIN32 OR NOT MSVC" OFF)
@@ -235,9 +236,14 @@ elseif (("${CMAKE_C_COMPILER_ID}" STREQUAL "GNU") OR ("${CMAKE_C_COMPILER_ID}" M
235236
set(CMAKE_SHARED_LIBRARY_LINK_C_FLAGS "")
236237
set(CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS "")
237238

238-
if(CMAKE_BUILD_TYPE MATCHES "DEBUG" OR CMAKE_BUILD_TYPE MATCHES "Debug" AND UseSanitizer)
239+
if(UseAddressSanitizer)
240+
# also raise stack size drastically (to 64MiB), since the sanitizer adds overhead to stack frames
239241
add_compile_options(-fsanitize=address)
240-
add_link_options(-fsanitize=address)
242+
add_link_options(-fsanitize=address -z stack-size=4000000)
243+
endif()
244+
if(UseUndefinedSanitizer)
245+
add_compile_options(-fsanitize=undefined)
246+
add_link_options(-fsanitize=undefined)
241247
endif()
242248

243249
# additional flags for debug configuration

0 commit comments

Comments
 (0)