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

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 21 09:29:37 PDT 2019


lebedev.ri added a comment.

Please upload all patches with full context (`-U99999`)



================
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)
----------------
Sure it isn't just the `SampleAnalyzer` that should be wrapped into `CLANG_BUILD_EXAMPLES`?


================
Comment at: clang/test/CMakeLists.txt:130
+  list(APPEND CLANG_TEST_DEPS
+    SampleAnalyzerPlugin
+    CheckerDependencyHandlingAnalyzerPlugin
----------------
Likewise


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