[MC] Simplify code with parseComma
authorFangrui Song <i@maskray.me>
Thu, 2 Feb 2023 02:41:02 +0000 (18:41 -0800)
committerFangrui Song <i@maskray.me>
Thu, 2 Feb 2023 02:41:02 +0000 (18:41 -0800)
llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
llvm/lib/Target/CSKY/AsmParser/CSKYAsmParser.cpp
llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
llvm/test/MC/ARM/directive-eabi_attribute-diagnostics.s
llvm/test/MC/ARM/thumb_set-diagnostics.s

index 671d263..b14abcb 100644 (file)
@@ -5609,7 +5609,7 @@ bool AMDGPUAsmParser::ParseDirectiveAMDGPULDS() {
     return TokError("expected identifier in directive");
 
   MCSymbol *Symbol = getContext().getOrCreateSymbol(Name);
-  if (parseToken(AsmToken::Comma, "expected ','"))
+  if (getParser().parseComma())
     return true;
 
   unsigned LocalMemorySize = AMDGPU::IsaInfo::getLocalMemorySize(&getSTI());
index 7d4fc97..b678fc5 100644 (file)
@@ -11422,7 +11422,7 @@ bool ARMAsmParser::parseDirectiveEabiAttr(SMLoc L) {
     Tag = CE->getValue();
   }
 
-  if (Parser.parseToken(AsmToken::Comma, "comma expected"))
+  if (Parser.parseComma())
     return true;
 
   StringRef StringValue = "";
@@ -11456,7 +11456,7 @@ bool ARMAsmParser::parseDirectiveEabiAttr(SMLoc L) {
   }
 
   if (Tag == ARMBuildAttrs::compatibility) {
-    if (Parser.parseToken(AsmToken::Comma, "comma expected"))
+    if (Parser.parseComma())
       return true;
   }
 
@@ -11668,7 +11668,7 @@ bool ARMAsmParser::parseDirectiveSetFP(SMLoc L) {
   int FPReg = tryParseRegister();
 
   if (check(FPReg == -1, FPRegLoc, "frame pointer register expected") ||
-      Parser.parseToken(AsmToken::Comma, "comma expected"))
+      Parser.parseComma())
     return true;
 
   // Parse spreg
@@ -11927,7 +11927,7 @@ bool ARMAsmParser::parseDirectiveUnwindRaw(SMLoc L) {
 
   StackOffset = CE->getValue();
 
-  if (Parser.parseToken(AsmToken::Comma, "expected comma"))
+  if (Parser.parseComma())
     return true;
 
   SmallVector<uint8_t, 16> Opcodes;
@@ -12071,7 +12071,7 @@ bool ARMAsmParser::parseDirectiveThumbSet(SMLoc L) {
   StringRef Name;
   if (check(Parser.parseIdentifier(Name),
             "expected identifier after '.thumb_set'") ||
-      parseToken(AsmToken::Comma, "expected comma after name '" + Name + "'"))
+      Parser.parseComma())
     return true;
 
   MCSymbol *Sym;
index 94ef40e..11ed3dd 100644 (file)
@@ -1660,7 +1660,7 @@ bool CSKYAsmParser::parseDirectiveAttribute() {
     Tag = CE->getValue();
   }
 
-  if (Parser.parseToken(AsmToken::Comma, "comma expected"))
+  if (Parser.parseComma())
     return true;
 
   StringRef StringValue;
index 1ade316..4835fdf 100644 (file)
@@ -2231,7 +2231,7 @@ bool RISCVAsmParser::parseDirectiveAttribute() {
     Tag = CE->getValue();
   }
 
-  if (Parser.parseToken(AsmToken::Comma, "comma expected"))
+  if (Parser.parseComma())
     return true;
 
   StringRef StringValue;
index 2b0375e..a20c630 100644 (file)
 @ CHECK:                            ^
 
        .eabi_attribute 0
-@ CHECK: error: comma expected
+@ CHECK: :[[#@LINE-1]]:19: error: expected comma
 @ CHECK:       .eabi_attribute 0
 @ CHECK:                         ^
 
         .eabi_attribute Tag_compatibility, 1
-@ CHECK: error: comma expected
+@ CHECK: :[[#@LINE-1]]:45: error: expected comma
 @ CHECK: .eabi_attribute Tag_compatibility, 1
 @ CHECK:                                     ^
 
index 4dd35d2..dedc9dd 100644 (file)
@@ -24,8 +24,7 @@
 @ CHECK:                            ^
 
        .thumb_set missing_comma
-
-@ CHECK: error: expected comma after name 'missing_comma'
+@ CHECK: :[[#@LINE-1]]:26: error: expected comma
 @ CHECK:       .thumb_set missing_comma
 @ CHECK:                                ^