Skip to content

Commit 0bd614a

Browse files
authored
[Cygwin] Don't use version script for Cygwin target (#143133)
Cygwin is a COFF platform and does not support version-script. I guess I should use LLVM_HAVE_LINK_VERSION_SCRIPT here, but I don't know why this is not currently the case.
1 parent 35e3c50 commit 0bd614a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/tools/llvm-shlib/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ if(LLVM_BUILD_LLVM_DYLIB)
6666
else()
6767
# GNU ld doesn't resolve symbols in the version script.
6868
set(LIB_NAMES -Wl,--whole-archive ${LIB_NAMES} -Wl,--no-whole-archive)
69-
if (NOT LLVM_LINKER_IS_SOLARISLD AND NOT MINGW)
69+
if (NOT LLVM_LINKER_IS_SOLARISLD AND NOT MINGW AND NOT CYGWIN)
7070
# Solaris ld does not accept global: *; so there is no way to version *all* global symbols
7171
set(LIB_NAMES -Wl,--version-script,${LLVM_LIBRARY_DIR}/tools/llvm-shlib/simple_version_script.map ${LIB_NAMES})
7272
endif()

0 commit comments

Comments
 (0)