[llvm] [ThinLTO] Properly support targets that require importing all external functions (PR #133588)
Yaxun Liu via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 31 06:12:43 PDT 2025
yxsamliu wrote:
> > Looks like this flag was added several years ago for AMDGPU (https://reviews.llvm.org/D99683). Do you know why it was sufficient then but not now?
>
> I'm not sure if ThinLTO actually worked on AMDGPU back then. At least, it doesn't seem to be working at the moment. I'd assume @yxsamliu might have more insight on this.
>
It worked for some HIP apps.
I am still not clear about what issue the linkage available_externally causes. Can you give an example? Thanks.
https://github.com/llvm/llvm-project/pull/133588
More information about the llvm-commits
mailing list