Skip to content

Commit 784d83b

Browse files
arsenmmemfrob
authored andcommitted
AMDGPU: Add assertion to determineCalleeSaves
Make sure this isn't getting called multiple times. I was surprised we were modifying the function here, which I think is a bit questionable.
1 parent 2479cf8 commit 784d83b

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

llvm/lib/Target/AMDGPU/SIFrameLowering.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1303,13 +1303,18 @@ void SIFrameLowering::determineCalleeSaves(MachineFunction &MF,
13031303
LiveRegs.init(*TRI);
13041304

13051305
if (WillHaveFP || hasFP(MF)) {
1306+
assert(!MFI->SGPRForFPSaveRestoreCopy && !MFI->FramePointerSaveIndex &&
1307+
"Re-reserving spill slot for FP");
13061308
getVGPRSpillLaneOrTempRegister(MF, LiveRegs, MFI->SGPRForFPSaveRestoreCopy,
13071309
MFI->FramePointerSaveIndex, true);
13081310
}
13091311

13101312
if (TRI->hasBasePointer(MF)) {
13111313
if (MFI->SGPRForFPSaveRestoreCopy)
13121314
LiveRegs.addReg(MFI->SGPRForFPSaveRestoreCopy);
1315+
1316+
assert(!MFI->SGPRForBPSaveRestoreCopy &&
1317+
!MFI->BasePointerSaveIndex && "Re-reserving spill slot for BP");
13131318
getVGPRSpillLaneOrTempRegister(MF, LiveRegs, MFI->SGPRForBPSaveRestoreCopy,
13141319
MFI->BasePointerSaveIndex, false);
13151320
}

0 commit comments

Comments
 (0)