[llvm] r361340 - [Analysis] Link library dependencies to Analysis plugins

[Petr Hosek via llvm-commits]

Author: phosek
Date: Tue May 21 17:47:37 2019
New Revision: 361340

URL: http://llvm.org/viewvc/llvm-project?rev=361340&view=rev
Log:
[Analysis] Link library dependencies to Analysis plugins

These are needed to avoid undefined symbols which aren't satisfied
by Clang itself.

Differential Revision: https://reviews.llvm.org/D62174

Modified:
    llvm/trunk/cmake/modules/HandleLLVMOptions.cmake

Modified: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?rev=361340&r1=361339&r2=361340&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/HandleLLVMOptions.cmake (original)
+++ llvm/trunk/cmake/modules/HandleLLVMOptions.cmake Tue May 21 17:47:37 2019
@@ -914,10 +914,14 @@ endif()
 
 # Plugin support
 # FIXME: Make this configurable.
-if(BUILD_SHARED_LIBS OR LLVM_BUILD_LLVM_DYLIB)
-  set(LLVM_ENABLE_PLUGINS ON)
+if(WIN32 OR CYGWIN)
+  if(BUILD_SHARED_LIBS OR LLVM_BUILD_LLVM_DYLIB)
+    set(LLVM_ENABLE_PLUGINS ON)
+  else()
+    set(LLVM_ENABLE_PLUGINS OFF)
+  endif()
 else()
-  set(LLVM_ENABLE_PLUGINS OFF)
+  set(LLVM_ENABLE_PLUGINS ON)
 endif()
 
 # By default we should enable LLVM_ENABLE_IDE only for multi-configuration