yxsamliu wrote:

@jhuber6 Re: device-only linker path

So far we have not received complaints about device-only compilation time being 
a problem, so I agree we can defer the `--offload-jobs` unification work.

I updated this PR to use the smaller `constructLldCommand` fix for the concrete 
profile-runtime issue. I kept `-amdgpu-internalize-symbols` unchanged since it 
is known to be critical for HIP app performance and was added for HIP 
compile-time/code-size/performance. If there is a correctness issue with it in 
LTO mode, could you share a concrete reproducer? I think that should be handled 
in a separate PR.

https://github.com/llvm/llvm-project/pull/211675
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to