[PATCH] D31026: [Asm] Don't list '@<type>' in diag when '@' is a comment

[Oliver Stannard via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL298067: [Asm] Don't list '@<type>' in diag when '@' is a comment (authored by olista01).

Changed prior to commit:
  https://reviews.llvm.org/D31026?vs=91983&id=92126#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D31026

Files:
  llvm/trunk/lib/MC/MCParser/ELFAsmParser.cpp
  llvm/trunk/test/MC/ELF/ARM/gnu-type-hash-diagnostics.s


Index: llvm/trunk/test/MC/ELF/ARM/gnu-type-hash-diagnostics.s
===================================================================
--- llvm/trunk/test/MC/ELF/ARM/gnu-type-hash-diagnostics.s
+++ llvm/trunk/test/MC/ELF/ARM/gnu-type-hash-diagnostics.s
@@ -7,3 +7,7 @@
 // CHECK: .type TYPE #32
 // CHECK:             ^
 
+  // For ARM, the comment character is '@', so we don't list '@<type>' as a
+  // valid option.
+  .section "foo", "a", @progbits
+// CHECK: error: expected '%<type>' or "<type>"
Index: llvm/trunk/lib/MC/MCParser/ELFAsmParser.cpp
===================================================================
--- llvm/trunk/lib/MC/MCParser/ELFAsmParser.cpp
+++ llvm/trunk/lib/MC/MCParser/ELFAsmParser.cpp
@@ -391,8 +391,12 @@
     return false;
   Lex();
   if (L.isNot(AsmToken::At) && L.isNot(AsmToken::Percent) &&
-      L.isNot(AsmToken::String))
-    return TokError("expected '@<type>', '%<type>' or \"<type>\"");
+      L.isNot(AsmToken::String)) {
+    if (L.getAllowAtInIdentifier())
+      return TokError("expected '@<type>', '%<type>' or \"<type>\"");
+    else
+      return TokError("expected '%<type>' or \"<type>\"");
+  }
   if (!L.is(AsmToken::String))
     Lex();
   if (L.is(AsmToken::Integer)) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31026.92126.patch
Type: text/x-patch
Size: 1224 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170317/7f8a8c63/attachment.bin>