This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Don't use estimated stack size when we know the real stack size
ClosedPublic

Authored by arsenm on Feb 29 2016, 4:00 PM.

Reviewers

• tstellarAMD

arsenm retitled this revision from to AMDGPU: Don't use estimated stack size when we know the real stack size.

arsenm updated this object.

arsenm added a subscriber: llvm-commits.

LGTM.

This revision is now accepted and ready to land.Feb 29 2016, 4:07 PM

r262297

Path

Size

lib/

Target/

AMDGPU/

2 lines

Show First 20 Lines • Show All 476 Lines • ▼ Show 20 Lines	void AMDGPUAsmPrinter::getSIProgramInfo(SIProgramInfo &ProgInfo,
ProgInfo.FloatMode = getFPMode(MF);		ProgInfo.FloatMode = getFPMode(MF);

ProgInfo.IEEEMode = 0;		ProgInfo.IEEEMode = 0;

// Make clamp modifier on NaN input returns 0.		// Make clamp modifier on NaN input returns 0.
ProgInfo.DX10Clamp = 1;		ProgInfo.DX10Clamp = 1;

const MachineFrameInfo *FrameInfo = MF.getFrameInfo();		const MachineFrameInfo *FrameInfo = MF.getFrameInfo();
ProgInfo.ScratchSize = FrameInfo->estimateStackSize(MF);		ProgInfo.ScratchSize = FrameInfo->getStackSize();

ProgInfo.FlatUsed = FlatUsed;		ProgInfo.FlatUsed = FlatUsed;
ProgInfo.VCCUsed = VCCUsed;		ProgInfo.VCCUsed = VCCUsed;
ProgInfo.CodeLen = CodeSize;		ProgInfo.CodeLen = CodeSize;

unsigned LDSAlignShift;		unsigned LDSAlignShift;
if (STM.getGeneration() < AMDGPUSubtarget::SEA_ISLANDS) {		if (STM.getGeneration() < AMDGPUSubtarget::SEA_ISLANDS) {
// LDS is allocated in 64 dword blocks.		// LDS is allocated in 64 dword blocks.
▲ Show 20 Lines • Show All 208 Lines • Show Last 20 Lines