[PATCH] D62050: [Analysis] Only run plugins tests if plugins are actually enabled

[Don Hinton via Phabricator via cfe-commits]

hintonda added inline comments.


================
Comment at: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake:930
-else()
+if(BUILD_SHARED_LIBS OR LLVM_BUILD_LLVM_DYLIB)
   set(LLVM_ENABLE_PLUGINS ON)
 endif()
----------------
This is a breaking patch.  What's the rational for unilaterally disabling LLVM_ENABLE_PLUGINS.

```
It gives the following errors:
-- PrintFunctionNames ignored -- Loadable modules not supported on this platform.
-- AnnotateFunctions ignored -- Loadable modules not supported on this platform.
-- SampleAnalyzerPlugin ignored -- Loadable modules not supported on this platform.
-- CheckerDependencyHandlingAnalyzerPlugin ignored -- Loadable modules not supported on this platform.
-- CheckerOptionHandlingAnalyzerPlugin ignored -- Loadable modules not supported on this platform.
-- BugpointPasses ignored -- Loadable modules not supported on this platform.
-- TestPlugin ignored -- Loadable modules not supported on this platform.
```


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62050