[Openmp-commits] [PATCH] D136911: [DeviceRTL] Fix incremental build

[Ye Luo via Phabricator via Openmp-commits]

ye-luo created this revision.
Herald added a project: All.
ye-luo requested review of this revision.
Herald added a project: OpenMP.
Herald added a subscriber: openmp-commits.

Need both add_custom_command to resolve file-level dependency and add_custom_target to resolve target-level dependency.
>From CMake add_custom_command doc:
Do not list the output in more than one independent target that may build in parallel or the two instances of the rule may conflict (instead use the add_custom_target() command to drive the command and make the other targets depend on that one).

${CMAKE_CURRENT_BINARY_DIR}/${bclib_name} is used by multiple targets and thus requires a custom target to avoid racing.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D136911

Files:
  openmp/libomptarget/DeviceRTL/CMakeLists.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136911.471379.patch
Type: text/x-patch
Size: 8719 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20221028/5e73ade4/attachment.bin>