[llvm] [AMDGPU] Support alloca in AS0 (PR #136584)

Shilei Tian via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 24 22:25:08 PDT 2025


shiltian wrote:

Based on the feedback in https://github.com/llvm/llvm-project/pull/136865 and https://github.com/llvm/llvm-project/pull/135820, I think not supporting this is the more appropriate direction. I propose that we close this PR and instead enforce that `alloca` must be in AS5 in the verifier, basically reverting to what https://github.com/llvm/llvm-project/pull/135820 initially did.

This way, we avoid hitting backend errors like "cannot select frameindex"” which can be misleading and make it sound like a backend bug when it actually isn't. If we find more practical use cases in the future that require supporting AS0, we can always revisit and reopen this PR.

What do you think? @arsenm

https://github.com/llvm/llvm-project/pull/136584


More information about the llvm-commits mailing list