[PATCH] D129507: [llvm-objcopy] Add option to dump embedded offloading images

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 16:15:57 PDT 2022


tra added a comment.

> The question isn't raised for the first time why we have so many llvm-*/clang-* tools doing GPU/offloading stuff now.

Well, it's a bit more complicated, but I do agree that offloading machinery we have is a bit of a mess and could use some streamlining.

That said, the issue at hand here is orthogonal to that. I'll take your word that llvm-objcopy is not the right place for offloading image manipulation.
That leaves some other tool. 
The options I see are:

- llvm-objdump -- may not be the best fit as we want to save offload binaries to files, not just print them out. Whether we may ever want to embed offloading binaries is an open question. I think it could be useful, but it's probably a very niche use case, even for those few of us who care what's under the hood in the offloading binaries.
- incorporate functionality into `clang-offload-packager`.
- write a new tool?

I personally would prefer to avoid creating yet another tool. clang-offload-packager may be a reasonable place for this, considering that it already deals with the new offloading binary packaging style.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129507



More information about the llvm-commits mailing list