Author: mgorny Date: Wed Mar 15 00:55:43 2017 New Revision: 297806 URL: http://llvm.org/viewvc/llvm-project?rev=297806&view=rev Log: [test] Fix test dependencies when using installed tools
Use the LLVM_UTILS_PROVIDED variable to determine whether test tool dependencies should be exposed for clang-tools-extra tests. If clang is being built stand-alone and LLVM test tools (FileCheck, count and not) are installed, the top-level CMakeLists.txt of clang sets this variable to indicate that they will not be built as a part of this build, and therefore no dependencies should be emitted for them. This fixes the dependency errors when building clang stand-alone with tests enabled. Differential Revision: https://reviews.llvm.org/D29851 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=297806&r1=297805&r2=297806&view=diff ============================================================================== --- clang-tools-extra/trunk/test/CMakeLists.txt (original) +++ clang-tools-extra/trunk/test/CMakeLists.txt Wed Mar 15 00:55:43 2017 @@ -31,9 +31,6 @@ if(CLANG_TOOLS_TEST_USE_VG) endif() set(CLANG_TOOLS_TEST_DEPS - # Base line deps. - FileCheck count not - # clang-tidy tests require it. clang-headers @@ -58,6 +55,13 @@ set(CLANG_TOOLS_TEST_DEPS ExtraToolsUnitTests ) +if(NOT LLVM_UTILS_PROVIDED) + list(APPEND CLANG_TOOLS_TEST_DEPS + # Base line deps. + FileCheck count not + ) +endif() + add_lit_testsuite(check-clang-tools "Running the Clang extra tools' regression tests" ${CMAKE_CURRENT_BINARY_DIR} DEPENDS ${CLANG_TOOLS_TEST_DEPS} _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits