This is an archive of the discontinued LLVM Phabricator instance.

R600/SI: Use register class instead of list of registers
ClosedPublic

Authored by arsenm on Aug 1 2014, 4:04 PM.

Download Raw Diff

Details

Reviewers

Summary

I'm not sure if this has any consequence or not. I noticed this while debugging what seems to be selecting the wrong super register class with SRegs sometimes

Diff Detail

Event Timeline

arsenm updated this revision to Diff 12123.Aug 1 2014, 4:04 PM

arsenm retitled this revision from to R600/SI: Use register class instead of list of registers .

arsenm updated this object.

arsenm edited the test plan for this revision. (Show Details)

arsenm added a subscriber: Unknown Object (MLST).

r214902

This revision is now accepted and ready to land.Aug 5 2014, 11:02 AM

arsenm closed this revision.Aug 5 2014, 11:02 AM

Revision Contents

Path

Size

lib/

Target/

R600/

SIRegisterInfo.td

2 lines

Diff 12123

lib/Target/R600/SIRegisterInfo.td

Context not available.
	def SGPR_64 : RegisterClass<"AMDGPU", [v2i32, i64], 64, (add SGPR_64Regs)>;	def SGPR_64 : RegisterClass<"AMDGPU", [v2i32, i64], 64, (add SGPR_64Regs)>;

	def SReg_64 : RegisterClass<"AMDGPU", [v2i32, i64, i1], 64,	def SReg_64 : RegisterClass<"AMDGPU", [v2i32, i64, i1], 64,
	(add SGPR_64Regs, VCCReg, EXECReg)	(add SGPR_64, VCCReg, EXECReg)
	>;	>;

	def SReg_128 : RegisterClass<"AMDGPU", [v4i32, v16i8], 128, (add SGPR_128)>;	def SReg_128 : RegisterClass<"AMDGPU", [v4i32, v16i8], 128, (add SGPR_128)>;
Context not available.