[Openmp-commits] [PATCH] D135630: [DeviceRTL] CMake fix using target-level dependency

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Oct 11 09:38:18 PDT 2022


tianshilei1992 added a comment.

We also need dependencies to those tools for each target.



================
Comment at: openmp/libomptarget/DeviceRTL/CMakeLists.txt:175
   )
+  add_dependencies(linked_${bclib_name} ${bc_files})
 
----------------
it occurs to me that, we also need to add `${LINK_TOOL}` to dependencies.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135630



More information about the Openmp-commits mailing list