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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to