libRemarks.{so,dylib}: remove Remarks.dylib.exports

Remarks.exports is only intended for NOT (BUILD_SHARED_LIBS OR LLVM_LINK_LLVM_DYLIB)
builds.

For (unintended use case) BUILD_SHARED_LIBS OR LLVM_LINK_LLVM_DYLIB (the latter
is used by some Linux distros), the library defines just one symbol on ELF.
There is no need to use a version script.

I think this is a more proper solution than D139932 and fixes `symbol not
defined` errors after lld default change D135402.
This commit is contained in:
Fangrui Song 2023-02-02 15:13:04 -08:00
parent c294bdd647
commit bf07de38b0
2 changed files with 1 additions and 4 deletions

View File

@ -9,9 +9,7 @@ if(LLVM_ENABLE_PIC)
libremarks.cpp
)
if (LLVM_LINK_LLVM_DYLIB)
set(LLVM_EXPORTED_SYMBOL_FILE ${CMAKE_CURRENT_SOURCE_DIR}/Remarks.dylib.exports)
else()
if (NOT (BUILD_SHARED_LIBS OR LLVM_LINK_LLVM_DYLIB))
set(LLVM_EXPORTED_SYMBOL_FILE ${CMAKE_CURRENT_SOURCE_DIR}/Remarks.exports)
endif()

View File

@ -1 +0,0 @@
LLVMRemarkVersion