[PATCH] D52334: [clang-tidy] Build it even without static analyzer

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 25 01:24:11 PDT 2018


steveire marked an inline comment as done.
steveire added inline comments.


================
Comment at: test/CMakeLists.txt:69
 
-    clang-tidy
-    )
-endif()
+  clang-tidy
+  )
----------------
alexfh wrote:
> There are some clang-tidy tests for the static analyzer integration (at least test/clang-tidy/static-analyzer.cpp and test/clang-tidy/static-analyzer-config.cpp). I would expect them to start failing when clang-tidy is built without static analyzer. Did you try running all the tests?
I realized that the tests were all excluded by the 'static-analyzer' switch too, so I've enabled them and disabled the ones which require the static-analyzer.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52334





More information about the cfe-commits mailing list