[llvm] [AMDGPU] Do not fold an immediate into instructions with frame indexes (PR #151263)

Changpeng Fang via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 3 23:23:47 PDT 2025


changpeng wrote:

> Missing a new test that would break. This looks too conservative; it will always be OK if the immediate is an inline immediate

Hi, @arsenm :  New test has been added. And we already filtered out the case of inline immediate, i.e. we can fold frame index into an instruction with an inline immediate operand.
Do you have any further suggestion?

Also do you think it is cheaper just to revert the offending patch: https://github.com/llvm/llvm-project/pull/140587

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


More information about the llvm-commits mailing list