[PATCH] D62873: Avoid building analyzer plugins if CLANG_ENABLE_STATIC_ANALYZER is OFF

Don Hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 4 16:01:50 PDT 2019


hintonda accepted this revision.
hintonda added a comment.
This revision is now accepted and ready to land.

LGTM...



================
Comment at: test/CMakeLists.txt:122
 
-if (CLANG_ENABLE_STATIC_ANALYZER)
-  if (LLVM_ENABLE_PLUGINS)
-    list(APPEND CLANG_TEST_DEPS
-      SampleAnalyzerPlugin
-      CheckerDependencyHandlingAnalyzerPlugin
-      CheckerOptionHandlingAnalyzerPlugin
-      )
-  endif()
+if(LLVM_ENABLE_PLUGINS AND CLANG_ENABLE_STATIC_ANALYZER)
+  list(APPEND CLANG_TEST_DEPS
----------------
I didn't fix this one, but makes sense. 


Repository:
  rC Clang

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

https://reviews.llvm.org/D62873





More information about the cfe-commits mailing list