Skip to content

Commit fe45029

Browse files
committed
[cmake] Back out of making unsupported -no_exported_symbols linker a
fatal error for now Appeases build bots while being investigated.
1 parent 80deb82 commit fe45029

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

llvm/cmake/modules/AddLLVM.cmake

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1038,14 +1038,9 @@ macro(add_llvm_executable name)
10381038
add_llvm_symbol_exports( ${name} ${LLVM_EXPORTED_SYMBOL_FILE} )
10391039
endif(LLVM_EXPORTED_SYMBOL_FILE)
10401040

1041-
if (NOT LLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUTABLES)
1042-
if(LLVM_LINKER_SUPPORTS_NO_EXPORTED_SYMBOLS)
1041+
if (NOT LLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUTABLES AND LLVM_LINKER_SUPPORTS_NO_EXPORTED_SYMBOLS)
10431042
set_property(TARGET ${name} APPEND_STRING PROPERTY
10441043
LINK_FLAGS " -Wl,-no_exported_symbols")
1045-
else()
1046-
message(FATAL_ERROR
1047-
"LLVM_ENABLE_EXPORTED_SYMBOLS_IN_EXECUTABLES cannot be disabled when linker does not support \"-no_exported_symbols\"")
1048-
endif()
10491044
endif()
10501045

10511046
if (LLVM_LINK_LLVM_DYLIB AND NOT ARG_DISABLE_LLVM_LINK_LLVM_DYLIB)

0 commit comments

Comments
 (0)