- Use a more appropriate name for Owen's ARM Parser isMCR hack since the same operands can be present
in cdp/cdp2 instructions. Also increase the hack with cdp/cdp2 instructions.
- Fix the encoding of cdp/cdp2 instructions for ARM (no thumb and thumb2 yet) and add testcases for t
hem.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@123927 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Target/ARM/AsmParser/ARMAsmParser.cpp b/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
index ba42b1e..a10e158 100644
--- a/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
+++ b/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
@@ -53,11 +53,11 @@
bool Error(SMLoc L, const Twine &Msg) { return Parser.Error(L, Msg); }
int TryParseRegister();
- bool TryParseMCRName(SmallVectorImpl<MCParsedAsmOperand*>&);
+ bool TryParseCoprocessorOperandName(SmallVectorImpl<MCParsedAsmOperand*>&);
bool TryParseRegisterWithWriteBack(SmallVectorImpl<MCParsedAsmOperand*> &);
bool ParseRegisterList(SmallVectorImpl<MCParsedAsmOperand*> &);
bool ParseMemory(SmallVectorImpl<MCParsedAsmOperand*> &);
- bool ParseOperand(SmallVectorImpl<MCParsedAsmOperand*> &, bool isMCR);
+ bool ParseOperand(SmallVectorImpl<MCParsedAsmOperand*> &, bool hasCoprocOp);
bool ParsePrefix(ARMMCExpr::VariantKind &RefKind);
const MCExpr *ApplyPrefixToExpr(const MCExpr *E,
MCSymbolRefExpr::VariantKind Variant);
@@ -602,7 +602,7 @@
return false;
}
-static int MatchMCRName(StringRef Name) {
+static int MatchCoprocessorOperandName(StringRef Name) {
// Use the same layout as the tablegen'erated register name matcher. Ugly,
// but efficient.
switch (Name.size()) {
@@ -643,17 +643,18 @@
return -1;
}
-/// TryParseMCRName - Try to parse an MCR/MRC symbolic operand
-/// name. The token must be an Identifier when called, and if it is a MCR
-/// operand name, the token is eaten and the operand is added to the
-/// operand list.
+/// TryParseCoprocessorOperandName - Try to parse an coprocessor related
+/// instruction with a symbolic operand name. The token must be an Identifier
+/// when called, and if it is a coprocessor related operand name, the token is
+/// eaten and the operand is added to the operand list. Example: operands like
+/// "p1", "p7", "c3", "c5", ...
bool ARMAsmParser::
-TryParseMCRName(SmallVectorImpl<MCParsedAsmOperand*> &Operands) {
+TryParseCoprocessorOperandName(SmallVectorImpl<MCParsedAsmOperand*> &Operands) {
SMLoc S = Parser.getTok().getLoc();
const AsmToken &Tok = Parser.getTok();
assert(Tok.is(AsmToken::Identifier) && "Token is not an Identifier");
- int Num = MatchMCRName(Tok.getString());
+ int Num = MatchCoprocessorOperandName(Tok.getString());
if (Num == -1)
return true;
@@ -966,7 +967,7 @@
/// Parse a arm instruction operand. For now this parses the operand regardless
/// of the mnemonic.
bool ARMAsmParser::ParseOperand(SmallVectorImpl<MCParsedAsmOperand*> &Operands,
- bool isMCR){
+ bool hasCoprocOp){
SMLoc S, E;
switch (getLexer().getKind()) {
default:
@@ -975,7 +976,7 @@
case AsmToken::Identifier:
if (!TryParseRegisterWithWriteBack(Operands))
return false;
- if (isMCR && !TryParseMCRName(Operands))
+ if (hasCoprocOp && !TryParseCoprocessorOperandName(Operands))
return false;
// Fall though for the Identifier case that is not a register or a
@@ -1264,15 +1265,22 @@
Operands.push_back(ARMOperand::CreateToken(Head, NameLoc));
}
- bool isMCR = (Head == "mcr" || Head == "mcr2" ||
- Head == "mcrr" || Head == "mcrr2" ||
- Head == "mrc" || Head == "mrc2" ||
- Head == "mrrc" || Head == "mrrc2");
+ // Enable the parsing of instructions containing coprocessor related
+ // asm syntax, such as coprocessor names "p7, p15, ..." and coprocessor
+ // registers "c1, c3, ..."
+ // FIXME: we probably want AsmOperandClass and ParserMatchClass declarations
+ // in the .td file rather than hacking the ASMParser for every symbolic
+ // operand type.
+ bool hasCoprocOp = (Head == "mcr" || Head == "mcr2" ||
+ Head == "mcrr" || Head == "mcrr2" ||
+ Head == "mrc" || Head == "mrc2" ||
+ Head == "mrrc" || Head == "mrrc2" ||
+ Head == "cdp" || Head == "cdp2");
// Read the remaining operands.
if (getLexer().isNot(AsmToken::EndOfStatement)) {
// Read the first operand.
- if (ParseOperand(Operands, isMCR)) {
+ if (ParseOperand(Operands, hasCoprocOp)) {
Parser.EatToEndOfStatement();
return true;
}
@@ -1281,7 +1289,7 @@
Parser.Lex(); // Eat the comma.
// Parse and remember the operand.
- if (ParseOperand(Operands, isMCR)) {
+ if (ParseOperand(Operands, hasCoprocOp)) {
Parser.EatToEndOfStatement();
return true;
}