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
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
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits