[clang] f0858bf - [Clang] Update Interpreter tests to use clang_target_link_libraries (#110154)

via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 2 00:12:20 PDT 2024


Author: Thomas Fransham
Date: 2024-10-02T10:12:17+03:00
New Revision: f0858bfb62468e5887e7b7d158bd07543ed82467

URL: https://github.com/llvm/llvm-project/commit/f0858bfb62468e5887e7b7d158bd07543ed82467
DIFF: https://github.com/llvm/llvm-project/commit/f0858bfb62468e5887e7b7d158bd07543ed82467.diff

LOG: [Clang] Update Interpreter tests to use clang_target_link_libraries (#110154)

This will fix duplicate and missing linker symbol errors when using
CLANG_LINK_CLANG_DYLIB on windows and explicit visibility macros are
used.
This is part of the work to enable LLVM_BUILD_LLVM_DYLIB and plugins on
window.

Added: 
    

Modified: 
    clang/unittests/Interpreter/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/clang/unittests/Interpreter/CMakeLists.txt b/clang/unittests/Interpreter/CMakeLists.txt
index ec6f81ea19b960..1ed1216c772e8f 100644
--- a/clang/unittests/Interpreter/CMakeLists.txt
+++ b/clang/unittests/Interpreter/CMakeLists.txt
@@ -16,13 +16,15 @@ add_clang_unittest(ClangReplInterpreterTests
 
   EXPORT_SYMBOLS
   )
-target_link_libraries(ClangReplInterpreterTests PUBLIC
+  
+target_link_libraries(ClangReplInterpreterTests PUBLIC LLVMTestingSupport)
+
+clang_target_link_libraries(ClangReplInterpreterTests PRIVATE
   clangAST
   clangBasic
   clangInterpreter
   clangFrontend
   clangSema
-  LLVMTestingSupport
   )
 
 # Exceptions on Windows are not yet supported.


        


More information about the cfe-commits mailing list