From: Aaron Puchert Date: Sun, 11 Sep 2022 18:44:51 +0000 (+0200) Subject: Make sure libLLVM users link with libatomic if needed X-Git-Tag: upstream/17.0.6~33833 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=44ffc13f2eb6188a86ae88ea1e942e9ac354db9b;p=platform%2Fupstream%2Fllvm.git Make sure libLLVM users link with libatomic if needed 64-bit atomics are used in llvm/ADT/Statistic.h, which means that users of libLLVM.so might also have to link with libatomic. To avoid having to special-case the library here, we simply add all `LLVM_SYSTEM_LIBS` as public link dependencies to libLLVM. This fixes a build failure on PowerPC 32-bit. Reviewed By: beanz Differential Revision: https://reviews.llvm.org/D132799 --- diff --git a/llvm/tools/llvm-shlib/CMakeLists.txt b/llvm/tools/llvm-shlib/CMakeLists.txt index 8e2b78f..cc6713a7 100644 --- a/llvm/tools/llvm-shlib/CMakeLists.txt +++ b/llvm/tools/llvm-shlib/CMakeLists.txt @@ -63,6 +63,9 @@ if(LLVM_BUILD_LLVM_DYLIB) target_link_libraries(LLVM PRIVATE ${LIB_NAMES}) + get_property(SYSTEM_LIBS TARGET LLVMSupport PROPERTY LLVM_SYSTEM_LIBS) + target_link_libraries(LLVM PUBLIC ${SYSTEM_LIBS}) + if (APPLE) set_property(TARGET LLVM APPEND_STRING PROPERTY LINK_FLAGS