This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Use CreateStackObject instead of CreateSpillStackObject
ClosedPublic

Authored by arsenm on Aug 4 2016, 2:46 PM.

Reviewers

• tstellarAMD

Summary

I'm not sure what the difference is, but no other target uses this for emergency spill slots.

arsenm retitled this revision from to AMDGPU: Use CreateStackObject instead of CreateSpillStackObject.

arsenm updated this object.

arsenm added a subscriber: llvm-commits.

LGTM.

This revision is now accepted and ready to land.Aug 5 2016, 4:37 PM

r278272

Path

Size

lib/

Target/

AMDGPU/

4 lines

Show First 20 Lines • Show All 284 Lines • ▼ Show 20 Lines	if (!MFI.hasStackObjects())
return;		return;

bool MayNeedScavengingEmergencySlot = MFI.hasStackObjects();		bool MayNeedScavengingEmergencySlot = MFI.hasStackObjects();

assert((RS \|\| !MayNeedScavengingEmergencySlot) &&		assert((RS \|\| !MayNeedScavengingEmergencySlot) &&
"RegScavenger required if spilling");		"RegScavenger required if spilling");

if (MayNeedScavengingEmergencySlot) {		if (MayNeedScavengingEmergencySlot) {
int ScavengeFI = MFI.CreateSpillStackObject(		int ScavengeFI = MFI.CreateStackObject(
AMDGPU::SGPR_32RegClass.getSize(),		AMDGPU::SGPR_32RegClass.getSize(),
AMDGPU::SGPR_32RegClass.getAlignment());		AMDGPU::SGPR_32RegClass.getAlignment(), false);
RS->addScavengingFrameIndex(ScavengeFI);		RS->addScavengingFrameIndex(ScavengeFI);
}		}
}		}

void SIFrameLowering::emitDebuggerPrologue(MachineFunction &MF,		void SIFrameLowering::emitDebuggerPrologue(MachineFunction &MF,
MachineBasicBlock &MBB) const {		MachineBasicBlock &MBB) const {
const SISubtarget &ST = MF.getSubtarget<SISubtarget>();		const SISubtarget &ST = MF.getSubtarget<SISubtarget>();
const SIInstrInfo *TII = ST.getInstrInfo();		const SIInstrInfo *TII = ST.getInstrInfo();
Show All 36 Lines