This is an archive of the discontinued LLVM Phabricator instance.

Differential D11884

AMDGPU/SI: Add missing spill class
ClosedPublic

Authored by axeldavy on Aug 9 2015, 5:59 AM.

Details

Reviewers
• tstellarAMD
arsenm
Summary

The compiler was failing to spill for some shaders.

Diff Detail

Event Timeline

axeldavy updated this revision to Diff 31616.Aug 9 2015, 5:59 AM
axeldavy retitled this revision from to AMDGPU/SI: Add missing spill class .
axeldavy updated this object.
axeldavy added a subscriber: llvm-commits.
axeldavy added reviewers: arsenm, • tstellarAMD.Aug 9 2015, 6:06 AM
• tstellarAMD accepted this revision.Aug 13 2015, 2:22 PM
• tstellarAMD edited edge metadata.

LGTM, I can commit this for you.

This revision is now accepted and ready to land.Aug 13 2015, 2:22 PM
axeldavy added a comment.Aug 13 2015, 10:12 PM

Yes sure, I do not have the rights to push it.

axeldavy closed this revision.Aug 16 2015, 6:44 AM

Revision Contents

PathSize
lib/
Target/
AMDGPU/
3 lines

Diff 31616

lib/Target/AMDGPU/SIRegisterInfo.cpp

Context not available.
&AMDGPU::SReg_128RegClass, &AMDGPU::SReg_128RegClass,
&AMDGPU::VReg_256RegClass, &AMDGPU::VReg_256RegClass,
&AMDGPU::SReg_256RegClass, &AMDGPU::SReg_256RegClass,
&AMDGPU::VReg_512RegClass &AMDGPU::VReg_512RegClass,
&AMDGPU::SReg_512RegClass
}; };
for (const TargetRegisterClass *BaseClass : BaseClasses) { for (const TargetRegisterClass *BaseClass : BaseClasses) {
Context not available.