[PATCH] D147654: [OpenMP] Add tool dependencies in runtimes mode

[Joseph Huber via Phabricator via llvm-commits]

jhuber6 created this revision.
jhuber6 added reviewers: tra, jdoerfert, ye-luo, tianshilei1992, JonChesterfield.
Herald added subscribers: sunshaoce, guansong, yaxunl.
Herald added a project: All.
jhuber6 requested review of this revision.
Herald added subscribers: llvm-commits, jplehr, sstefan1.
Herald added a project: LLVM.

The OpenMP offloading device runtime depends on multiple tools. We
should ensure that these tools are build before invoking the OpenMP
build.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D147654

Files:
  llvm/runtimes/CMakeLists.txt


Index: llvm/runtimes/CMakeLists.txt
===================================================================
--- llvm/runtimes/CMakeLists.txt
+++ llvm/runtimes/CMakeLists.txt
@@ -382,12 +382,11 @@
   # together in a single CMake invocation.
   set(extra_deps "")
   if("openmp" IN_LIST LLVM_ENABLE_RUNTIMES)
-    if(TARGET opt)
-      list(APPEND extra_deps opt)
-    endif()
-    if(TARGET llvm-link)
-      list(APPEND extra_deps llvm-link)
-    endif()
+    foreach(dep opt llvm-link llvm-extract clang clang-offload-packager)
+      if(TARGET ${dep} AND OPENMP_ENABLE_LIBOMPTARGET)
+        list(APPEND extra_deps ${dep})
+      endif()
+    endforeach()
   endif()
   if("libc" IN_LIST LLVM_ENABLE_RUNTIMES AND 
       (LLVM_LIBC_FULL_BUILD OR LIBC_GPU_BUILD OR LIBC_GPU_ARCHITECTURES))


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147654.511184.patch
Type: text/x-patch
Size: 787 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230405/ecfc6062/attachment.bin>