From 50ec033305f1c471924846dcf66c0e15e54c5ffc Mon Sep 17 00:00:00 2001 From: Michael Kruse Date: Thu, 27 Apr 2017 16:13:03 +0000 Subject: [PATCH] [CMake] Use object library to build the two flavours of Polly. Polly comes in two library flavors: One loadable module to use the LLVM framework -load mechanism, and another one that host applications can link to. These have very different requirements for Polly's own dependencies. The loadable module assumes that all its LLVM dependencies are already available in the address space of the host application, and is not allowed to bring in its own copy of any LLVM library (including the NVPTX backend in case of Polly-ACC). The non-module library is intended to be linked to using target_link_libraries. CMake would then resolve all of its dependencies, including NVPTX and ensure that only a single instance of each library will be used. Differential Revision: https://reviews.llvm.org/D32442 git-svn-id: https://llvm.org/svn/llvm-project/polly/trunk@301558 91177308-0d34-0410-b5e6-96231b3b80d8 --- CMakeLists.txt | 2 +- lib/CMakeLists.txt | 110 +++++++++++++++++++++++++++++++++-------------- unittests/CMakeLists.txt | 15 +------ 3 files changed, 81 insertions(+), 46 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index a5db3589..c3e06602 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -104,7 +104,7 @@ if (NOT DEFINED LLVM_MAIN_SRC_DIR) "${UNITTEST_DIR}/googletest" "${UNITTEST_DIR}/googlemock" ) - target_link_libraries(gtest ${LLVM_SYSTEM_LIBS}) + target_link_libraries(gtest -lpthread) add_library(gtest_main ${UNITTEST_DIR}/UnitTestMain/TestMain.cpp) target_link_libraries(gtest_main gtest) diff --git a/lib/CMakeLists.txt b/lib/CMakeLists.txt index 8842ca91..63c76463 100644 --- a/lib/CMakeLists.txt +++ b/lib/CMakeLists.txt @@ -26,7 +26,9 @@ if (MSVC_IDE OR XCODE) file(GLOB_RECURSE POLLY_HEADER_FILES "${POLLY_SOURCE_DIR}/include/polly/*.h") endif () -add_polly_library(Polly +# Use an object-library to add the same files to multiple libs without requiring +# the sources them to be recompiled for each of them. +add_library(PollyCore OBJECT Analysis/DependenceInfo.cpp Analysis/PolyhedralInfo.cpp Analysis/ScopDetection.cpp @@ -66,44 +68,76 @@ add_polly_library(Polly ${POLLY_HEADER_FILES} ) +# Create the library that can be linked into LLVM's tools and Polly's unittests. +# It depends on all library it needs, such that with +# LLVM_POLLY_LINK_INTO_TOOLS=ON, its dependencies like PollyISL are linked as +# well. +add_polly_library(Polly $) +target_link_libraries(Polly + ${ISL_TARGET} +) + +# Additional dependencies for Polly-ACC. if (GPU_CODEGEN) target_link_libraries(Polly PollyPPCG) -endif (GPU_CODEGEN) +endif () -target_link_libraries(Polly ${ISL_TARGET}) +# Add Polly's LLVM dependencies. +# When built inside the LLVM source tree, these are CMake targets that will +# depend on their dependencies and CMake ensures they are added in the right +# order. +# If Polly is built independently, just add all LLVM libraries. LLVM_ROOT_DIR +# might have been configured to compile to individual libraries or a single +# libLLVM.so (called dylib), reported by llvm-config, so there is no fixed list +# of required libraries. +if (DEFINED LLVM_MAIN_SRC_DIR) -if (BUILD_SHARED_LIBS) - target_link_libraries(Polly - LLVMSupport - LLVMCore - LLVMScalarOpts - LLVMInstCombine - LLVMTransformUtils - LLVMAnalysis - LLVMipo - LLVMMC + # Polly-ACC requires the NVPTX backend to work. Ask LLVM about its libraries. + set(nvptx_libs) + if (GPU_CODEGEN) + # This call emits an error if they NVPTX backend is not enable. + llvm_map_components_to_libnames(nvptx_libs NVPTX) + endif () + + if (LLVM_LINK_LLVM_DYLIB) + # The shlib/dylib contains all the LLVM components + # (including NVPTX is enabled) already. Adding them to target_link_libraries + # would cause them being twice in the address space + # (their LLVM*.a/so and their copies in libLLVM.so) + # which results in errors when the two instances try to register the same + # command-line switches. + target_link_libraries(Polly LLVM) + else () + target_link_libraries(Polly + LLVMSupport + LLVMCore + LLVMScalarOpts + LLVMInstCombine + LLVMTransformUtils + LLVMAnalysis + LLVMipo + LLVMMC + ${nvptx_libs} # The libraries below are required for darwin: http://PR26392 - LLVMBitReader - LLVMMCParser - LLVMObject - LLVMProfileData - LLVMTarget - LLVMVectorize - ) - link_directories( - ${LLVM_LIBRARY_DIR} - ) -elseif (LLVM_LINK_LLVM_DYLIB) + LLVMBitReader + LLVMMCParser + LLVMObject + LLVMProfileData + LLVMTarget + LLVMVectorize + ) + endif () +else () + # When Polly-ACC is enabled, we assume that the host LLVM was also built with + # the NVPTX target enabled. target_link_libraries(Polly - LLVM + ${LLVM_LIBS} + ${LLVM_SYSTEM_LIBS} ) - link_directories( - ${LLVM_LIBRARY_DIR} - ) -endif() +endif () -# Build a monolithic Polly.a and a thin module LLVMPolly.moduleext that links to -# that static library. +# Create a loadable module Polly.so that can be loaded using +# LLVM's/clang's "-load" option. if (MSVC) # Add dummy target, because loadable modules are not supported on Windows add_custom_target(LLVMPolly) @@ -111,9 +145,21 @@ if (MSVC) else () add_polly_loadable_module(LLVMPolly Polly.cpp + $ ) - target_link_libraries(LLVMPolly Polly) + # Only add the dependencies that are not part of LLVM. The latter are assumed + # to be already available in the address space the module is loaded into. + # Adding them once more would have the effect that both copies try to register + # the same command line options, to which LLVM reacts with an error. + # If Polly-ACC is enabled, the NVPTX target is also expected to reside in the + # hosts. This is not the case for bugpoint. Use LLVM_POLLY_LINK_INTO_TOOLS=ON + # instead which will automatically resolve the additional dependencies by + # Polly. + target_link_libraries(LLVMPolly ${ISL_TARGET}) + if (GPU_CODEGEN) + target_link_libraries(LLVMPolly PollyPPCG) + endif () set_target_properties(LLVMPolly PROPERTIES diff --git a/unittests/CMakeLists.txt b/unittests/CMakeLists.txt index 1dfc3097..e5e0f6c5 100644 --- a/unittests/CMakeLists.txt +++ b/unittests/CMakeLists.txt @@ -7,27 +7,16 @@ set_target_properties(PollyUnitTests PROPERTIES FOLDER "Polly") function(add_polly_unittest test_name) if(COMMAND add_unittest) add_unittest(PollyUnitTests ${test_name} ${ARGN}) - target_link_libraries(${test_name} Polly) - - # The Polly target does not depend on its required libraries, except: - # - BUILD_SHARED_LIBS=ON - # in which case calling target_link_libraries again is redundant. - # - LLVM_LINK_LLVM_DYLIB=ON - # in which case it depends on libLLVM.so, so no additional libs needed. - # We are also not allowed to link against the plain LLVM* libraries, - # which would result in multiple instances of these to be loaded. - if (NOT LLVM_LINK_LLVM_DYLIB) - target_link_libraries(${test_name} LLVMCore LLVMSupport LLVMDemangle LLVMipo) - endif () else() add_executable(${test_name} EXCLUDE_FROM_ALL ${ARGN}) set_target_properties(${test_name} PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}) - target_link_libraries(${test_name} gtest_main gtest Polly ${LLVM_LIBS}) + target_link_libraries(${test_name} gtest_main gtest) add_dependencies(PollyUnitTests ${test_name}) set_property(TARGET ${test_name} PROPERTY FOLDER "Polly") endif() + target_link_libraries(${test_name} Polly) endfunction() add_subdirectory(Isl) -- 2.11.4.GIT