Skip to content

Commit 59eb2fe

Browse files
committed
[llvm] Enable building Analysis plugins on Cygwin (llvm#151397)
In llvm#112303 they were enabled for Windows dylib builds, but the condition excluded Cygwin in that case. Their dynamic libraries work the same way, so adjust the condition accordingly. This fixes the plugin tests on Cygwin.
1 parent 794856e commit 59eb2fe

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

llvm/unittests/Analysis/InlineAdvisorPlugin/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# libraries, but expects them to exist in the process loading the plugin. This
33
# doesn't work with DLLs on Windows (where a shared library can't have undefined
44
# references), so just skip this testcase on Windows.
5-
if ((NOT WIN32 OR LLVM_BUILD_LLVM_DYLIB) AND NOT CYGWIN)
5+
if ((NOT WIN32 AND NOT CYGWIN) OR LLVM_BUILD_LLVM_DYLIB)
66
unset(LLVM_LINK_COMPONENTS)
77
add_llvm_library(InlineAdvisorPlugin MODULE BUILDTREE_ONLY
88
InlineAdvisorPlugin.cpp

llvm/unittests/Analysis/InlineOrderPlugin/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# libraries, but expects them to exist in the process loading the plugin. This
33
# doesn't work with DLLs on Windows (where a shared library can't have undefined
44
# references), so just skip this testcase on Windows.
5-
if ((NOT WIN32 OR LLVM_BUILD_LLVM_DYLIB) AND NOT CYGWIN)
5+
if ((NOT WIN32 AND NOT CYGWIN) OR LLVM_BUILD_LLVM_DYLIB)
66
unset(LLVM_LINK_COMPONENTS)
77
add_llvm_library(InlineOrderPlugin MODULE BUILDTREE_ONLY
88
InlineOrderPlugin.cpp

0 commit comments

Comments
 (0)