Make the wording in of the "expected identifier" error in the .macro directive
consistent with the other "expected identifier" errors. Extracted from the Andy/PaX patch. I added the test. llvm-svn: 162291
This commit is contained in:
parent
5e80ced3f6
commit
af6da83a2c
|
@ -3071,14 +3071,14 @@ bool GenericAsmParser::ParseDirectiveMacro(StringRef Directive,
|
|||
SMLoc DirectiveLoc) {
|
||||
StringRef Name;
|
||||
if (getParser().ParseIdentifier(Name))
|
||||
return TokError("expected identifier in directive");
|
||||
return TokError("expected identifier in '.macro' directive");
|
||||
|
||||
MacroParameters Parameters;
|
||||
if (getLexer().isNot(AsmToken::EndOfStatement)) {
|
||||
for (;;) {
|
||||
MacroParameter Parameter;
|
||||
if (getParser().ParseIdentifier(Parameter))
|
||||
return TokError("expected identifier in directive");
|
||||
return TokError("expected identifier in '.macro' directive");
|
||||
Parameters.push_back(Parameter);
|
||||
|
||||
if (getLexer().isNot(AsmToken::Comma))
|
||||
|
|
|
@ -0,0 +1,9 @@
|
|||
// RUN: not llvm-mc -triple x86_64-apple-darwin10 %s 2>&1 | FileCheck %s
|
||||
|
||||
.macro 23
|
||||
|
||||
// CHECK: expected identifier in '.macro' directive
|
||||
|
||||
.macro abc 33
|
||||
|
||||
// CHECK: expected identifier in '.macro' directive
|
Loading…
Reference in New Issue