[PATCH] D52781: [clangd] Don't make check-clangd as a dependency in check-clang-tools
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 2 07:33:29 PDT 2018
hokein created this revision.
hokein added a reviewer: sammccall.
Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov, mgorny.
check-clang-tools will run check-clangd first, and then run the rest
tests. If clangd tests fails, check-clang-tools would be stopped.
This would block other clang-tools developers if clangd is broken.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52781
Files:
test/CMakeLists.txt
Index: test/CMakeLists.txt
===================================================================
--- test/CMakeLists.txt
+++ test/CMakeLists.txt
@@ -52,8 +52,6 @@
modularize
pp-trace
- check-clangd
-
# Unit tests
ExtraToolsUnitTests
@@ -65,6 +63,19 @@
clang-tidy
)
+set(CLANGD_TEST_DEPS
+ clangd
+ ClangdTests
+ # clangd-related tools which don't have tests, add them to the test to make
+ # sure we don't introduce new changes that break their compilations.
+ clangd-indexer
+ dexp
+ )
+foreach(clangd_dep ${CLANGD_TEST_DEPS})
+ list(APPEND CLANG_TOOLS_TEST_DEPS
+ ${clangd_dep})
+endforeach()
+
add_lit_testsuite(check-clang-tools "Running the Clang extra tools' regression tests"
${CMAKE_CURRENT_BINARY_DIR}
DEPENDS ${CLANG_TOOLS_TEST_DEPS}
@@ -74,15 +85,9 @@
set_target_properties(check-clang-tools PROPERTIES FOLDER "Clang extra tools' tests")
# Setup an individual test for building and testing clangd-only stuff.
-set(CLANGD_TEST_DEPS
- clangd
- ClangdTests
- # clangd-related tools which don't have tests, add them to the test to make
- # sure we don't introduce new changes that break their compilations.
- clangd-indexer
- dexp
-)
-# Exclude check-clangd from check-all, as check-clangd will launch via check-clang-tools.
+# Note: all clangd tests have been covered in check-clang-tools, this is a
+# convenient target for clangd developers.
+# Exclude check-clangd from check-all.
set(EXCLUDE_FROM_ALL ON)
add_lit_testsuite(check-clangd "Running the Clangd regression tests"
${CMAKE_CURRENT_BINARY_DIR}/Unit/clangd;${CMAKE_CURRENT_BINARY_DIR}/clangd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52781.167945.patch
Type: text/x-patch
Size: 1620 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181002/e587b4c3/attachment.bin>
More information about the cfe-commits
mailing list