[llvm] [AMDGPU] Move kernarg preload logic to AMDGPU Attributor (PR #123547)
Shilei Tian via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 7 11:57:27 PST 2025
shiltian wrote:
> I had a PR up to make AMDGPULowerKernelArguments a module pass, but I think the consensus was that it made sense to unify all the inreg/preload kernarg functionality here.
I remember that PR. I think that should be the way to go.
IIUC, @arsenm did mention in https://github.com/llvm/llvm-project/pull/101609 (which is indeed something about `inreg`) that we can probably move the logic _there_, but at that time I thought it runs part of the attributor iterations. I'm not sure if that is a good idea to pile everything in this file.
https://github.com/llvm/llvm-project/pull/123547
More information about the llvm-commits
mailing list