[llvm] r330329 - [Unittests] Fix plugins test

Mikhail Maltsev via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 19 07:02:47 PDT 2018


Author: miyuki
Date: Thu Apr 19 07:02:46 2018
New Revision: 330329

URL: http://llvm.org/viewvc/llvm-project?rev=330329&view=rev
Log:
[Unittests] Fix plugins test

Summary:
Currently the PluginsTests.LoadPlugin unit test is failing in
LLVM configurations that have LLVM_EXPORT_SYMBOLS_FOR_PLUGINS enabled
because the EnableABIBreakingChecks symbol is missing.

This patch fixes the issue by linking some additional libraries to the
test plugin if LLVM_EXPORT_SYMBOLS_FOR_PLUGINS is enabled.

Reviewers: philip.pfaffe

Reviewed By: philip.pfaffe

Subscribers: mgorny, llvm-commits, rogfer01

Differential Revision: https://reviews.llvm.org/D45811

Modified:
    llvm/trunk/unittests/Passes/CMakeLists.txt

Modified: llvm/trunk/unittests/Passes/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Passes/CMakeLists.txt?rev=330329&r1=330328&r2=330329&view=diff
==============================================================================
--- llvm/trunk/unittests/Passes/CMakeLists.txt (original)
+++ llvm/trunk/unittests/Passes/CMakeLists.txt Thu Apr 19 07:02:46 2018
@@ -16,7 +16,7 @@ if (LLVM_ENABLE_PLUGINS)
     SUFFIX ".so"
     )
 
-  if (WIN32 OR CYGWIN)
+  if (WIN32 OR CYGWIN OR LLVM_EXPORT_SYMBOLS_FOR_PLUGINS)
     llvm_map_components_to_libnames(LLVM_DEPS ${LLVM_LINK_COMPONENTS})
     target_link_libraries(TestPlugin ${LLVM_DEPS})
   endif()




More information about the llvm-commits mailing list