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

Stephen Kelly via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 22 09:05:08 PDT 2018


steveire added a comment.

Thanks, that at least makes it more obvious where you are getting confused.

See `tools/clang/lib/CMakeLists.txt`. It contains:

add_subdirectory(Analysis)

...
===

if(CLANG_ENABLE_STATIC_ANALYZER)

  add_subdirectory(StaticAnalyzer)

endif()

1. That is: Analysis and StaticAnalyzer are different.

CLANG_ENABLE_STATIC_ANALYZER refers to the latter.

2. Also, something that may not have occurred to you: This patch does build. Obviously, if there were a problem with CFG after this patch, then it would not build.

Please keep both of those points in mind as you read the patch.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D52334





More information about the cfe-commits mailing list