[PATCH] D147646: [compiler-rt] Create the umbrella check-compiler-rt test target in non-standalone builds too
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 5 11:36:07 PDT 2023
mstorsjo created this revision.
mstorsjo added reviewers: phosek, alvinhochun, vitalybuka.
Herald added subscribers: Enna1, dberris.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: Sanitizers.
Herald added a subscriber: Sanitizers.
I don't see why we can't have this target when built as a project alongside
llvm; especially in such build configs, it's useful to have one single target
for running all tests (in standalone builds, check-all does the same too),
Keep the creation of the check-all target only for standalone builds.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D147646
Files:
compiler-rt/test/CMakeLists.txt
Index: compiler-rt/test/CMakeLists.txt
===================================================================
--- compiler-rt/test/CMakeLists.txt
+++ compiler-rt/test/CMakeLists.txt
@@ -43,9 +43,7 @@
endif()
endif()
-if(COMPILER_RT_STANDALONE_BUILD)
- umbrella_lit_testsuite_begin(check-compiler-rt)
-endif()
+umbrella_lit_testsuite_begin(check-compiler-rt)
function(compiler_rt_test_runtime runtime)
string(TOUPPER ${runtime} runtime_uppercase)
@@ -110,12 +108,12 @@
add_subdirectory(shadowcallstack)
endif()
-if(COMPILER_RT_STANDALONE_BUILD)
- # Now that we've traversed all the directories and know all the lit testsuites,
- # introduce a rule to run to run all of them.
- add_custom_target(compiler-rt-test-depends DEPENDS ${LLVM_COMPILER_RT_LIT_DEPENDS})
- umbrella_lit_testsuite_end(check-compiler-rt)
+# Now that we've traversed all the directories and know all the lit testsuites,
+# introduce a rule to run to run all of them.
+add_custom_target(compiler-rt-test-depends DEPENDS ${LLVM_COMPILER_RT_LIT_DEPENDS})
+umbrella_lit_testsuite_end(check-compiler-rt)
+if(COMPILER_RT_STANDALONE_BUILD)
if(NOT TARGET check-all)
add_custom_target(check-all)
endif()
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147646.511171.patch
Type: text/x-patch
Size: 1197 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230405/0420e18c/attachment.bin>
More information about the llvm-commits
mailing list