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

[Ye Luo via Phabricator via Openmp-commits]

ye-luo added inline comments.


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


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135630