From b547fc76bb73636dca67943d2fdb5d721491659f Mon Sep 17 00:00:00 2001 From: Petr Hosek Date: Fri, 17 May 2019 06:07:37 +0000 Subject: [PATCH] [Analysis] Only run plugins tests if plugins are actually enabled 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 | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/cmake/modules/HandleLLVMOptions.cmake b/cmake/modules/HandleLLVMOptions.cmake index ada9a680649..f716dbdcd2e 100644 --- a/cmake/modules/HandleLLVMOptions.cmake +++ b/cmake/modules/HandleLLVMOptions.cmake @@ -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 -- 2.50.1