Commit 92d1195b authored by Matt Arsenault's avatar Matt Arsenault
Browse files

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.
parent 7c03f7d7

llvm/lib/Target/AMDGPU/SIFrameLowering.cpp

+5 −0
Original line number Diff line number Diff line
@@ -1303,6 +1303,8 @@ void SIFrameLowering::determineCalleeSaves(MachineFunction &MF, 
  LiveRegs.init(*TRI);



  if (WillHaveFP || hasFP(MF)) {

    assert(!MFI->SGPRForFPSaveRestoreCopy && !MFI->FramePointerSaveIndex &&

           "Re-reserving spill slot for FP");

    getVGPRSpillLaneOrTempRegister(MF, LiveRegs, MFI->SGPRForFPSaveRestoreCopy,

                                   MFI->FramePointerSaveIndex, true);

  }
@@ -1310,6 +1312,9 @@ void SIFrameLowering::determineCalleeSaves(MachineFunction &MF, 
  if (TRI->hasBasePointer(MF)) {

    if (MFI->SGPRForFPSaveRestoreCopy)

      LiveRegs.addReg(MFI->SGPRForFPSaveRestoreCopy);



    assert(!MFI->SGPRForBPSaveRestoreCopy &&

           !MFI->BasePointerSaveIndex && "Re-reserving spill slot for BP");

    getVGPRSpillLaneOrTempRegister(MF, LiveRegs, MFI->SGPRForBPSaveRestoreCopy,

                                   MFI->BasePointerSaveIndex, false);

  }