This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][AsmParser] Add isScalarReg() and repurpose isReg()
ClosedPublic

Authored by sdesmalen on Dec 20 2017, 6:40 AM.

Details

Summary

isReg() in AArch64AsmParser.cpp is a bit of a misnomer, and would be better named 'isScalarReg()' instead.

Patch [1/3] in a series to add operand constraint checks for SVE's predicated ADD/SUB.

Diff Detail

Event Timeline

sdesmalen created this revision.Dec 20 2017, 6:40 AM
fhahn accepted this revision.Dec 21 2017, 3:17 AM

LGTM. MCParsedAsmOperand::isReg should return true if the operand is a register operand, not only for scalar registers.

This revision is now accepted and ready to land.Dec 21 2017, 3:17 AM
sdesmalen closed this revision.Jan 2 2018, 5:40 AM