Author: Martin Storsjö Date: 2021-08-02T21:25:29+03:00 New Revision: 332ac2f8794610c3afdb11099a32ec36382cb500
URL: https://github.com/llvm/llvm-project/commit/332ac2f8794610c3afdb11099a32ec36382cb500 DIFF: https://github.com/llvm/llvm-project/commit/332ac2f8794610c3afdb11099a32ec36382cb500.diff LOG: [clang-repl] Fix building with win32 dylibs Use `clang_target_link_libraries` to avoid duplicate libraries when the same symbol is provided both by a static library and a larger dylib, fixing linking with win32 dylibs. This fixes errors like these: ld.lld: error: duplicate symbol: llvm::createStringError(std::__1::error_code, char const*) >>> defined at libLLVMSupport.a(Error.cpp.obj) >>> defined at libLLVM-14git.dll This matches how other clang tools declare their dependencies. Differential Revision: https://reviews.llvm.org/D107231 (cherry picked from commit 25a288b009f7d30b5392ea36c29570cbdcf238c3) Added: Modified: clang/tools/clang-repl/CMakeLists.txt Removed: ################################################################################ diff --git a/clang/tools/clang-repl/CMakeLists.txt b/clang/tools/clang-repl/CMakeLists.txt index a9dcd1f6a66bb..ae0e4f39be70f 100644 --- a/clang/tools/clang-repl/CMakeLists.txt +++ b/clang/tools/clang-repl/CMakeLists.txt @@ -10,7 +10,7 @@ add_clang_executable(clang-repl ClangRepl.cpp ) -target_link_libraries(clang-repl PUBLIC +clang_target_link_libraries(clang-repl PUBLIC clangBasic clangInterpreter clangTooling _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits