Skip to content

Commit bc38c06

Browse files
committed
[Offload] Remove unnecessary omp CMake target dependencies
In my quest to build `liboffload` with ASAN enabled I've been looking into decoupling `offload` and `openmp` project entablement. When removing `openmp` from `LLVM_ENABLE_PROJECTS` it resulted in failure to link against the `omp` target. As an experiment I disabled that dependency and the build appears to work as expect. This patch removes dependencies on the `omp` target from `libomptarget`/`llvm-offload-device-info`/`llvm-omp-kernel-replay` targets.
1 parent b22b103 commit bc38c06

File tree

3 files changed

+0
-12
lines changed

3 files changed

+0
-12
lines changed

offload/libomptarget/CMakeLists.txt

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,5 @@
11
message(STATUS "Building offloading runtime library libomptarget.")
22

3-
if(LIBOMP_STANDALONE)
4-
set(LIBOMP ${LIBOMP_STANDALONE})
5-
else()
6-
set(LIBOMP omp)
7-
endif()
8-
93
add_llvm_library(omptarget
104
SHARED
115

@@ -33,10 +27,6 @@ add_llvm_library(omptarget
3327
Support
3428
Object
3529

36-
LINK_LIBS
37-
PUBLIC
38-
${LIBOMP}
39-
4030
NO_INSTALL_RPATH
4131
BUILDTREE_ONLY
4232
)

offload/tools/deviceinfo/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,5 @@ target_include_directories(llvm-offload-device-info PRIVATE
88
${LIBOMPTARGET_INCLUDE_DIR}
99
)
1010
target_link_libraries(llvm-offload-device-info PRIVATE
11-
omp
1211
omptarget
1312
)

offload/tools/kernelreplay/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,5 @@ target_include_directories(llvm-omp-kernel-replay PRIVATE
99
)
1010
target_link_libraries(llvm-omp-kernel-replay PRIVATE
1111
LLVMSupport
12-
omp
1312
omptarget
1413
)

0 commit comments

Comments
 (0)