r359739 - Revert r359717, "Make check-clang depend on the clang-check binary always"

Nico Weber via cfe-commits cfe-commits at lists.llvm.org
Wed May 1 16:32:38 PDT 2019


Author: nico
Date: Wed May  1 16:32:38 2019
New Revision: 359739

URL: http://llvm.org/viewvc/llvm-project?rev=359739&view=rev
Log:
Revert r359717, "Make check-clang depend on the clang-check binary always"

The Tooling tests do have a lit.local.cfg with

    if not config.root.clang_staticanalyzer:
        config.unsupported = True

so what's wrong isn't the missing dep, but that lit prints a warning for
the binary missing. This will need a different kind of fix.

Modified:
    cfe/trunk/test/CMakeLists.txt

Modified: cfe/trunk/test/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CMakeLists.txt?rev=359739&r1=359738&r2=359739&view=diff
==============================================================================
--- cfe/trunk/test/CMakeLists.txt (original)
+++ cfe/trunk/test/CMakeLists.txt Wed May  1 16:32:38 2019
@@ -49,7 +49,6 @@ list(APPEND CLANG_TEST_DEPS
   c-index-test
   clang
   clang-resource-headers
-  clang-check
   clang-format
   clang-tblgen
   clang-offload-bundler
@@ -63,6 +62,7 @@ list(APPEND CLANG_TEST_DEPS
   
 if(CLANG_ENABLE_STATIC_ANALYZER)
   list(APPEND CLANG_TEST_DEPS
+    clang-check
     clang-extdef-mapping
     )
 endif()




More information about the cfe-commits mailing list