build: add enable-assertions option

Chilledheart · Oct 20, 2024 · fd3e992 · fd3e992
1 parent e6d2f30
commit fd3e992
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -643,6 +643,18 @@ else()
   message(STATUS "Debug Build")
 endif()
 
+if (upper_CMAKE_BUILD_TYPE MATCHES "RELWITHDEBINFO|DEBUG" OR NOT OFFICIAL_BUILD)
+  option(ENABLE_ASSERTIONS "Enable assertions" ON)
+else()
+  option(ENABLE_ASSERTIONS "Enable assertions" OFF)
+endif()
+
+if (ENABLE_ASSERTIONS)
+  message(STATUS "Enabled Assertions")
+  list(APPEND YASS_APP_FEATURES "assertions")
+  add_definitions(-DDCHECK_ALWAYS_ON=1)
+endif()
+
 if (ENABLE_LTO AND IS_RELEASE_BUILD)
   if (COMPILER_CLANG)
     if (USE_MOLD AND UNIX AND NOT APPLE)

diff --git a/third_party/libc++/__assertion_handler b/third_party/libc++/__assertion_handler
@@ -18,7 +18,7 @@
 #endif
 
 // Based on CHECK_WILL_STREAM() defined in base/check.h
-#if defined(NDEBUG) && defined(OFFICIAL_BUILD)
+#if defined(OFFICIAL_BUILD) && !defined(DCHECK_ALWAYS_ON)
 
 _LIBCPP_BEGIN_NAMESPACE_STD