[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