[PATCH] D67877: Conditionnaly include clang Analysis examples with cmake.

Jean-Bapiste Lepesme via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 21 09:35:11 PDT 2019


Jiboo added inline comments.


================
Comment at: clang/lib/Analysis/plugins/CMakeLists.txt:1-2
-if(CLANG_ENABLE_STATIC_ANALYZER AND LLVM_ENABLE_PLUGINS)
+if(CLANG_ENABLE_STATIC_ANALYZER AND LLVM_ENABLE_PLUGINS AND CLANG_BUILD_EXAMPLES)
   add_subdirectory(SampleAnalyzer)
   add_subdirectory(CheckerDependencyHandling)
----------------
lebedev.ri wrote:
> Sure it isn't just the `SampleAnalyzer` that should be wrapped into `CLANG_BUILD_EXAMPLES`?
Dunno, I though they were all examples, due to their contents:

CheckerDependencyHandling: registry.addDependency("example.DependendentChecker", "example.Dependency");
CheckerOptionHandling: registry.addChecker(registerMyChecker, shouldRegisterMyChecker, "example.MyChecker", "Example Description",


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67877/new/

https://reviews.llvm.org/D67877





More information about the cfe-commits mailing list