[Openmp-commits] [PATCH] D139371: [OpenMP] Use `add_llvm_library` to build the target `PluginInterface` in `plugins-nextgen`

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 updated this revision to Diff 480498.
tianshilei1992 added a comment.

fix configure error


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139371/new/

https://reviews.llvm.org/D139371

Files:
  openmp/libomptarget/plugins-nextgen/common/PluginInterface/CMakeLists.txt
  openmp/libomptarget/plugins/common/elf_common/CMakeLists.txt


Index: openmp/libomptarget/plugins/common/elf_common/CMakeLists.txt
===================================================================
--- openmp/libomptarget/plugins/common/elf_common/CMakeLists.txt
+++ openmp/libomptarget/plugins/common/elf_common/CMakeLists.txt
@@ -10,21 +10,23 @@
 #
 ##===----------------------------------------------------------------------===##
 
-add_library(elf_common OBJECT elf_common.cpp ELFSymbols.cpp)
+add_llvm_library(elf_common elf_common.cpp ELFSymbols.cpp
+  BUILDTREE_ONLY
+
+  LINK_COMPONENTS
+    BinaryFormat
+    Object
+    Support
+
+  LINK_LIBS
+    ${OPENMP_PTHREAD_LIB}
+)
 
 # Build elf_common with PIC to be able to link it with plugin shared libraries.
 set_property(TARGET elf_common PROPERTY POSITION_INDEPENDENT_CODE ON)
-llvm_update_compile_flags(elf_common)
-set(LINK_LLVM_LIBS LLVMBinaryFormat LLVMObject LLVMSupport)
-if (LLVM_LINK_LLVM_DYLIB)
-  set(LINK_LLVM_LIBS LLVM)
-endif()
-target_link_libraries(elf_common INTERFACE ${LINK_LLVM_LIBS})
-add_dependencies(elf_common ${LINK_LLVM_LIBS})
-
-# The code uses Debug.h, which requires threads support.
-target_link_libraries(elf_common INTERFACE ${OPENMP_PTHREAD_LIB})
 
 # Expose elf_common.h directory to the users of this library.
-target_include_directories(elf_common INTERFACE ${CMAKE_CURRENT_SOURCE_DIR})
-target_include_directories(elf_common PRIVATE ${LIBOMPTARGET_INCLUDE_DIR})
+target_include_directories(elf_common
+  INTERFACE ${CMAKE_CURRENT_SOURCE_DIR}
+  PRIVATE ${LIBOMPTARGET_INCLUDE_DIR}
+)
Index: openmp/libomptarget/plugins-nextgen/common/PluginInterface/CMakeLists.txt
===================================================================
--- openmp/libomptarget/plugins-nextgen/common/PluginInterface/CMakeLists.txt
+++ openmp/libomptarget/plugins-nextgen/common/PluginInterface/CMakeLists.txt
@@ -11,24 +11,39 @@
 ##===----------------------------------------------------------------------===##
 
 # Plugin Interface library.
-add_library(PluginInterface OBJECT PluginInterface.cpp GlobalHandler.cpp)
+add_llvm_library(PluginInterface PluginInterface.cpp GlobalHandler.cpp
+  BUILDTREE_ONLY
 
-# Define the TARGET_NAME.
-add_definitions("-DTARGET_NAME=PluginInterface")
+  LINK_COMPONENTS
+    Support
 
-# Define the DEBUG_PREFIX.
-add_definitions(-DDEBUG_PREFIX="PluginInterface")
+  LINK_LIBS
+    elf_common
+)
+
+# NOTE: Please don't move `MemoryManager` to `LINK_LIBS` above, even though it
+# is an interface "library". Don't expect it to be treated as a "library" such
+# that we just need to link it via CMake and its include path will be updated
+# automatically. It is because `PluginInterface` is a static library here, and
+# per CMake design (which makes sense), privately linked libraries have to be
+# exported. However, there is no a typical way in CMake to export an interface
+# library without install rules. We apparently don't need `MemoryManager`'s
+# header out of the build tree.
+target_include_directories(PluginInterface PRIVATE
+  $<TARGET_PROPERTY:MemoryManager,INTERFACE_INCLUDE_DIRECTORIES>
+)
+
+# Define the TARGET_NAME and DEBUG_PREFIX.
+target_compile_definitions(PluginInterface PRIVATE
+  TARGET_NAME="PluginInterface"
+  DEBUG_PREFIX="PluginInterface"
+)
+
+target_include_directories(PluginInterface
+  INTERFACE ${CMAKE_CURRENT_SOURCE_DIR}
+  PRIVATE ${LIBOMPTARGET_INCLUDE_DIR}
+)
 
 set_target_properties(PluginInterface PROPERTIES
   POSITION_INDEPENDENT_CODE ON
   CXX_VISIBILITY_PRESET protected)
-llvm_update_compile_flags(PluginInterface)
-set(LINK_LLVM_LIBS LLVMSupport)
-if (LLVM_LINK_LLVM_DYLIB)
-  set(LINK_LLVM_LIBS LLVM)
-endif()
-target_link_libraries(PluginInterface INTERFACE ${LINK_LLVM_LIBS} PRIVATE elf_common MemoryManager)
-add_dependencies(PluginInterface ${LINK_LLVM_LIBS})
-
-target_include_directories(PluginInterface INTERFACE ${CMAKE_CURRENT_SOURCE_DIR})
-target_include_directories(PluginInterface PRIVATE ${LIBOMPTARGET_INCLUDE_DIR})


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D139371.480498.patch
Type: text/x-patch
Size: 3934 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20221206/9215a78f/attachment.bin>