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

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 24 07:21:21 PDT 2018


alexfh added a comment.

> ! In https://reviews.llvm.org/D52334#1242955, @JonasToth wrote:
>  ... to me it makes sense to have clang-tidy without CSA.

Yep, it seems reasonable.



================
Comment at: test/CMakeLists.txt:69
 
-    clang-tidy
-    )
-endif()
+  clang-tidy
+  )
----------------
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?


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52334





More information about the cfe-commits mailing list