This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Make flat_scr name consistent
ClosedPublic

Authored by arsenm on Oct 26 2015, 2:09 PM.

Reviewers

Summary

The printed name and the parsed assembler names weren't the same.
I'm not sure which name SC prints these as, but I think it's this one.

arsenm retitled this revision from to AMDGPU: Make flat_scr name consistent.

arsenm updated this object.

arsenm added a subscriber: llvm-commits.

Invert because sc does it the other way

LGTM.

This revision is now accepted and ready to land.Nov 2 2015, 8:18 PM

r252010

Path

Size

lib/

Target/

AMDGPU/

AsmParser/

6 lines

Show First 20 Lines • Show All 479 Lines • ▼ Show 20 Lines	static int getRegClass(bool IsVgpr, unsigned RegWidth) {
}		}
}		}

static unsigned getRegForName(StringRef RegName) {		static unsigned getRegForName(StringRef RegName) {

return StringSwitch<unsigned>(RegName)		return StringSwitch<unsigned>(RegName)
.Case("exec", AMDGPU::EXEC)		.Case("exec", AMDGPU::EXEC)
.Case("vcc", AMDGPU::VCC)		.Case("vcc", AMDGPU::VCC)
.Case("flat_scr", AMDGPU::FLAT_SCR)		.Case("flat_scratch", AMDGPU::FLAT_SCR)
.Case("m0", AMDGPU::M0)		.Case("m0", AMDGPU::M0)
.Case("scc", AMDGPU::SCC)		.Case("scc", AMDGPU::SCC)
.Case("flat_scr_lo", AMDGPU::FLAT_SCR_LO)		.Case("flat_scratch_lo", AMDGPU::FLAT_SCR_LO)
.Case("flat_scr_hi", AMDGPU::FLAT_SCR_HI)		.Case("flat_scratch_hi", AMDGPU::FLAT_SCR_HI)
.Case("vcc_lo", AMDGPU::VCC_LO)		.Case("vcc_lo", AMDGPU::VCC_LO)
.Case("vcc_hi", AMDGPU::VCC_HI)		.Case("vcc_hi", AMDGPU::VCC_HI)
.Case("exec_lo", AMDGPU::EXEC_LO)		.Case("exec_lo", AMDGPU::EXEC_LO)
.Case("exec_hi", AMDGPU::EXEC_HI)		.Case("exec_hi", AMDGPU::EXEC_HI)
.Default(0);		.Default(0);
}		}

bool AMDGPUAsmParser::ParseRegister(unsigned &RegNo, SMLoc &StartLoc, SMLoc &EndLoc) {		bool AMDGPUAsmParser::ParseRegister(unsigned &RegNo, SMLoc &StartLoc, SMLoc &EndLoc) {
▲ Show 20 Lines • Show All 1,290 Lines • Show Last 20 Lines