diff options
| author | Saleem Abdulrasool <compnerd@compnerd.org> | 2014-03-17 17:13:57 +0000 |
|---|---|---|
| committer | Saleem Abdulrasool <compnerd@compnerd.org> | 2014-03-17 17:13:57 +0000 |
| commit | 3f44cd7926eda24cce75746094fac58c2a4f4c92 (patch) | |
| tree | 15c0beb0fc891bc6a832257c2bd4573f7fedf79d /llvm/lib/MC/MCParser/AsmParser.cpp | |
| parent | 11543a9953111cd29623653d5f9ca5749f05b8d6 (diff) | |
| download | bcm5719-llvm-3f44cd7926eda24cce75746094fac58c2a4f4c92.tar.gz bcm5719-llvm-3f44cd7926eda24cce75746094fac58c2a4f4c92.zip | |
MC: change runtime check to an assertion
It is unclear how it would be possible to get M to be NULL in normal scenarios.
Change this to an assert rather than a runtime check as per dblakie's
suggestion.
llvm-svn: 204060
Diffstat (limited to 'llvm/lib/MC/MCParser/AsmParser.cpp')
| -rw-r--r-- | llvm/lib/MC/MCParser/AsmParser.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/MC/MCParser/AsmParser.cpp b/llvm/lib/MC/MCParser/AsmParser.cpp index fbbd4365bcb..665e3d33d88 100644 --- a/llvm/lib/MC/MCParser/AsmParser.cpp +++ b/llvm/lib/MC/MCParser/AsmParser.cpp @@ -1998,9 +1998,10 @@ bool AsmParser::parseMacroArguments(const MCAsmMacro *M, break; if (FAI >= NParameters) { + assert(M && "expected macro to be defined"); Error(IDLoc, "parameter named '" + FA.Name + "' does not exist for macro '" + - (M ? M->Name : "<unnamed>") + "'"); + M->Name + "'"); return true; } PI = FAI; |
