[libcxx-commits] [PATCH] D97913: [runtimes] Use add_lit_testsuite to register lit testsuites

Petr Hosek via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Mar 3 22:42:33 PST 2021


phosek created this revision.
phosek added reviewers: ldionne, smeenai.
Herald added a subscriber: mgorny.
phosek requested review of this revision.
Herald added projects: Sanitizers, libc++, libc++abi.
Herald added subscribers: libcxx-commits, Sanitizers.
Herald added a reviewer: libc++.
Herald added a reviewer: libc++abi.

The runtimes build uses variables set by add_lit_testsuite to collect
testsuites from all the runtimes.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D97913

Files:
  compiler-rt/test/CMakeLists.txt
  libcxx/test/CMakeLists.txt
  libcxxabi/test/CMakeLists.txt


Index: libcxxabi/test/CMakeLists.txt
===================================================================
--- libcxxabi/test/CMakeLists.txt
+++ libcxxabi/test/CMakeLists.txt
@@ -76,7 +76,7 @@
   ${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg
   MAIN_CONFIG "${CMAKE_CURRENT_SOURCE_DIR}/lit.cfg.py")
 
-add_lit_target(check-cxxabi "Running libcxxabi tests"
+add_lit_testsuite(check-cxxabi "Running libcxxabi tests"
   ${CMAKE_CURRENT_BINARY_DIR}
   DEPENDS ${LIBCXXABI_TEST_DEPS}
   PARAMS "${LIBCXXABI_TEST_PARAMS}")
Index: libcxx/test/CMakeLists.txt
===================================================================
--- libcxx/test/CMakeLists.txt
+++ libcxx/test/CMakeLists.txt
@@ -103,7 +103,7 @@
     DEPENDS cxx ${LIBCXX_TEST_DEPS}
     COMMENT "Builds dependencies required to run the test suite.")
 
-  add_lit_target(check-cxx
+  add_lit_testsuite(check-cxx
     "Running libcxx tests"
     ${CMAKE_CURRENT_BINARY_DIR}
     DEPENDS cxx-test-depends
Index: compiler-rt/test/CMakeLists.txt
===================================================================
--- compiler-rt/test/CMakeLists.txt
+++ compiler-rt/test/CMakeLists.txt
@@ -87,7 +87,7 @@
   # introduce a rule to run to run all of them.
   get_property(LLVM_LIT_TESTSUITES GLOBAL PROPERTY LLVM_LIT_TESTSUITES)
   get_property(LLVM_LIT_DEPENDS GLOBAL PROPERTY LLVM_LIT_DEPENDS)
-  add_lit_target(check-compiler-rt
+  add_lit_testsuite(check-compiler-rt
     "Running all regression tests"
     ${LLVM_LIT_TESTSUITES}
     DEPENDS ${LLVM_LIT_DEPENDS})


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D97913.328026.patch
Type: text/x-patch
Size: 1510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210304/c6b023c3/attachment.bin>


More information about the libcxx-commits mailing list