commit 5e00e5a20cb55e86e63f57325400d4b9ce272ba9
author Michael Kruse <llvm-project@meinersbur.de> Tue Aug 24 02:49:06 2021 -0500
committer Copybara-Service <copybara-worker@google.com> Tue Aug 24 01:10:36 2021 -0700
tree ed882b35f57de04b319c9c14422f3c7618aefcc9
parent d88f00a95a389b87cbb5ff5ea2fe9ab87ef979d0

[Polly] Don't redundantly link libPolly into unittests.

With LLVM_LINK_LLVM_DYLIB and LLVM_POLLY_LINK_INTO_TOOLS, Polly is
already linked into libLLVM.so, linking libPolly.a as well into
unittests results in duplicate command line registration errors.

GitOrigin-RevId: cdbc86dd225063beb9fc8f6ba1f09afc18435331

unittests/CMakeLists.txt

diff --git a/unittests/CMakeLists.txt b/unittests/CMakeLists.txt
index fac7038..94b7544 100644
--- a/unittests/CMakeLists.txt
+++ b/unittests/CMakeLists.txt
@@ -16,7 +16,13 @@
 
     set_property(TARGET ${test_name} PROPERTY FOLDER "Polly")
   endif()
-  target_link_libraries(${test_name} PRIVATE Polly)
+
+  if(LLVM_LINK_LLVM_DYLIB AND LLVM_POLLY_LINK_INTO_TOOLS)
+    # In this case Polly is already present in libLLVM,
+    # no need to link it again.
+  else()
+    target_link_libraries(${test_name} PRIVATE Polly)
+  endif()
 endfunction()
 
 add_subdirectory(Isl)