]> granicus.if.org Git - llvm/commitdiff
[Analysis] Only run plugins tests if plugins are actually enabled
authorPetr Hosek <phosek@chromium.org>
Fri, 17 May 2019 06:07:37 +0000 (06:07 +0000)
committerPetr Hosek <phosek@chromium.org>
Fri, 17 May 2019 06:07:37 +0000 (06:07 +0000)
When plugins aren't enabled, don't try to run plugins tests. Don't
enable plugins unconditionally based on the platform, instead check
if LLVM shared library is actually being built which may not be the
case for every host configuration, even if the host itself supports
plugins.

This addresses test failures introduced by r360891/D59464.

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@360991 91177308-0d34-0410-b5e6-96231b3b80d8

cmake/modules/HandleLLVMOptions.cmake

index ada9a680649b407d2616ed32b3b5bdd146c43653..f716dbdcd2eac7673336c90bdee25e4aec3afc87 100644 (file)
@@ -920,14 +920,10 @@ endif()
 
 # Plugin support
 # FIXME: Make this configurable.
-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()
+if(BUILD_SHARED_LIBS OR LLVM_BUILD_LLVM_DYLIB)
   set(LLVM_ENABLE_PLUGINS ON)
+else()
+  set(LLVM_ENABLE_PLUGINS OFF)
 endif()
 
 # By default we should enable LLVM_ENABLE_IDE only for multi-configuration