[llvm] [AMDGPU] Mark AGPR tuple implicit in the first instr of AGPR spills. (PR #115285)

Christudasan Devadasan via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 10 20:52:42 PST 2024


================
@@ -1808,6 +1808,8 @@ void SIRegisterInfo::buildSpillLoadStore(
                            .addReg(SubReg, getKillRegState(IsKill));
         if (NeedSuperRegDef)
           AccRead.addReg(ValueReg, RegState::ImplicitDefine);
+        if (NeedSuperRegImpOperand && (IsFirstSubReg || IsLastSubReg))
----------------
cdevadas wrote:

If adding the implicit operand to all the subregs is the right approach (as currently happening in the copyPhysReg), we should fix all the paths in this function to include the same for all sub-register spill instructions.

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


More information about the llvm-commits mailing list