yaxunl added a comment.

This may break -save-temps since the input to clang-offload-bundler may not be 
temporary files when -save-temps is enabled.

I think clang-offload-bundler is not the right place to decide whether a file 
is a temporary file. Whether a file is a temporary file should be determined at 
its point of creation and if it is a temporary file it should be addTempFile 
there, instead of guessing that later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107668

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to