[libc++][libc++abi] Serialize the enable_assertions Lit parameter in the generated...
authorLouis Dionne <ldionne.2@gmail.com>
Mon, 28 Mar 2022 17:10:14 +0000 (13:10 -0400)
committerLouis Dionne <ldionne.2@gmail.com>
Tue, 29 Mar 2022 12:17:25 +0000 (08:17 -0400)
This means that re-running with llvm-lit in that configuration will
work as expected. This also enables assertions in libc++abi in the
Generic-assertions CI job, which was disabled previously.

Differential Revision: https://reviews.llvm.org/D122597

libcxx/cmake/caches/Generic-assertions.cmake
libcxx/test/CMakeLists.txt
libcxxabi/test/CMakeLists.txt

index 18eb866..a1d348e 100644 (file)
@@ -1,3 +1,2 @@
 set(LIBCXX_ENABLE_ASSERTIONS ON CACHE BOOL "")
-set(LIBCXX_TEST_PARAMS "enable_assertions=True" CACHE STRING "")
-set(LIBCXXABI_TEST_PARAMS "enable_assertions=True" CACHE STRING "")
+set(LIBCXXABI_ENABLE_ASSERTIONS ON CACHE BOOL "")
index 1c9445b..ebabc70 100644 (file)
@@ -104,6 +104,10 @@ if (NOT LIBCXX_ENABLE_DEBUG_MODE_SUPPORT)
   serialize_lit_param(enable_debug_tests False)
 endif()
 
+if (LIBCXX_ENABLE_ASSERTIONS)
+  serialize_lit_param(enable_assertions True)
+endif()
+
 if (CMAKE_CXX_COMPILER_TARGET)
   serialize_lit_param(target_triple "\"${CMAKE_CXX_COMPILER_TARGET}\"")
 else()
index 608e5f3..c695069 100644 (file)
@@ -79,6 +79,10 @@ if (NOT LIBCXXABI_ENABLE_EXCEPTIONS)
   serialize_lit_param(enable_exceptions False)
 endif()
 
+if (LIBCXXABI_ENABLE_ASSERTIONS)
+  serialize_lit_param(enable_assertions True)
+endif()
+
 serialize_lit_param(enable_experimental False)
 
 if (LLVM_USE_SANITIZER)