Revert "build: add enable-assertions option"

This reverts commit 0ea8f8c.
Chilledheart · Oct 21, 2024 · b4616bd · b4616bd
1 parent 2cfaaf6
commit b4616bd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -643,18 +643,6 @@ 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(OFFICIAL_BUILD) && !defined(DCHECK_ALWAYS_ON)
+#if defined(NDEBUG) && defined(OFFICIAL_BUILD)
 
 _LIBCPP_BEGIN_NAMESPACE_STD