Author: hokein Date: Mon Oct 1 07:02:02 2018 New Revision: 343474 URL: http://llvm.org/viewvc/llvm-project?rev=343474&view=rev Log: [clangd] Add "check-clangd" target
Summary: So we don't have to run "check-clang-tools" (which builds and tests all clang tools) to verify our clangd-related change. It'd save waiting time for clangd developers. check-clangd (build ~1000 files, run ~340 tests) vs check-clang-tools (build ~3000 files, run ~1000 tests). In the future, we probably want to add similar target for other clang-tools (e.g. clang-tidy). Reviewers: sammccall Subscribers: mgorny, ilya-biryukov, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits Differential Revision: https://reviews.llvm.org/D52710 Modified: clang-tools-extra/trunk/test/CMakeLists.txt Modified: clang-tools-extra/trunk/test/CMakeLists.txt URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/CMakeLists.txt?rev=343474&r1=343473&r2=343474&view=diff ============================================================================== --- clang-tools-extra/trunk/test/CMakeLists.txt (original) +++ clang-tools-extra/trunk/test/CMakeLists.txt Mon Oct 1 07:02:02 2018 @@ -43,7 +43,6 @@ set(CLANG_TOOLS_TEST_DEPS # Individual tools we test. clang-apply-replacements clang-change-namespace - clangd clang-doc clang-include-fixer clang-move @@ -53,10 +52,7 @@ set(CLANG_TOOLS_TEST_DEPS modularize pp-trace - # These individual tools have no tests, add them here to make them compile - # together with check-clang-tools, so that we won't break them in the future. - clangd-indexer - dexp + check-clangd # Unit tests ExtraToolsUnitTests @@ -73,16 +69,6 @@ if(CLANG_ENABLE_STATIC_ANALYZER) ) endif() -set(llvm_utils - FileCheck count not - ) - -foreach(t ${llvm_utils}) - if(TARGET ${t}) - list(APPEND CLANG_TOOLS_TEST_DEPS ${t}) - endif() -endforeach() - add_lit_testsuite(check-clang-tools "Running the Clang extra tools' regression tests" ${CMAKE_CURRENT_BINARY_DIR} DEPENDS ${CLANG_TOOLS_TEST_DEPS} @@ -90,3 +76,18 @@ add_lit_testsuite(check-clang-tools "Run ) 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 +) +add_lit_testsuite(check-clangd "Running the Clangd regression tests" + ${CMAKE_CURRENT_BINARY_DIR}/Unit/clangd;${CMAKE_CURRENT_BINARY_DIR}/clangd + DEPENDS ${CLANGD_TEST_DEPS} +) +set_target_properties(check-clangd PROPERTIES FOLDER "Clangd tests") _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
