[PATCH] D59316: [HIP-Clang] propagate -mllvm options to opt and llc

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 15 12:14:14 PDT 2019

yaxunl added a comment.

In D59316#1431238 <https://reviews.llvm.org/D59316#1431238>, @arsenm wrote:

> In D59316#1429580 <https://reviews.llvm.org/D59316#1429580>, @yaxunl wrote:
> > Here we are looking at the code which emulates a "linker" for HIP toolchain. The offloading action builder requests the offloading toolchain have a linker, but amdgpu does not have a real linker (ISA level linker), so we have to emulate that. If we have an ISA level linker we can get rid of all these stuff, but I don't think that will happen in short time.
> This isn't really true. We do run lld to link the final executable. It also doesn't change that opt and llc should never be involved in the process

Can lld do ISA level linking? That is, one device function in one object file calls another device function in a different object file, and we let lld link them together?

  rC Clang



More information about the cfe-commits mailing list