[PATCH] D107668: [OpenMP]Fix PR50336: Remove temporary files in the offload bundler tool

Victor Lomuller via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 21 02:09:57 PDT 2022


Naghasan added a comment.
Herald added a subscriber: MaskRay.
Herald added a project: All.

Any rational for not calling `addTempFile` during the binding rather than the call setup ? I had a brief look there and it seems to me this is the root of the problem.

I got bitten by this patch in a downstream project where some of the files in the offload action are actual inputs and so got deleted.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107668



More information about the cfe-commits mailing list