thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-19 16:20:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't enable JEMALLOC_DEBUG on debug builds

Browse Source
This commit is contained in:
Raúl Marín 2024-08-27 19:29:42 +02:00
parent 08f5ecbf60
commit 49ee8d4109
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
contrib/jemalloc-cmake/CMakeLists.txt
View File

@ -164,14 +164,16 @@ target_compile_definitions(_jemalloc PRIVATE -DJEMALLOC_NO_PRIVATE_NAMESPACE)
# Because our coverage callbacks call malloc, and recursive call of malloc could not work.
target_compile_options(_jemalloc PRIVATE ${WITHOUT_COVERAGE_FLAGS_LIST})
if (CMAKE_BUILD_TYPE_UC STREQUAL "DEBUG")
target_compile_definitions(_jemalloc PRIVATE
-DJEMALLOC_DEBUG=1
# Usage examples:
# - MALLOC_CONF=log:.
# - MALLOC_CONF='log:core.malloc.exit|core.sallocx.entry|core.sdallocx.entry'
-DJEMALLOC_LOG=1)
endif ()
# We don't enable Jemalloc DEBUG and log anymore since those have a big impact in performance, specially noticeable
# when starting the process (really common in stateless tests with the client)
#if (CMAKE_BUILD_TYPE_UC STREQUAL "DEBUG")
# target_compile_definitions(_jemalloc PRIVATE
# -DJEMALLOC_DEBUG=1
# # Usage examples:
# # - MALLOC_CONF=log:.
# # - MALLOC_CONF='log:core.malloc.exit|core.sallocx.entry|core.sdallocx.entry'
# -DJEMALLOC_LOG=1)
#endif ()
target_compile_definitions(_jemalloc PRIVATE -DJEMALLOC_PROF=1)