[PATCH] D125165: [Clang] Introduce clang-offload-packager tool to bundle device files

[Joseph Huber via Phabricator via cfe-commits]

jhuber6 added a comment.

In D125165#3506172 <https://reviews.llvm.org/D125165#3506172>, @thakis wrote:

> We now have clang-offload-bundler, clang-offload-packager, clang-offload-wrapper. Do we really need that many distinct binaries for offloading? Any chance we could combine some of those?

Yeah, it's not ideal. I would've liked to use `clang-offload-bundler` for this tool but we're keeping the old ones around for backwards compatibility. The `clang-offload-bundler` and `clang-offload-wrapper` have already been merged into `clang` and the `clang-linker-wrapper` respectively, they're kept around for backwards compatibility currently. I think HIP still uses the `clang-offload-bundler` but I'm in the process of changing that. I'm not sure if it's possible to delete these tools once they're introduced since someone might rely on the binary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125165