diff options
| author | Chris Lattner <sabre@nondot.org> | 2010-09-11 16:18:25 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2010-09-11 16:18:25 +0000 |
| commit | cbf8a98c7c652e96967623c80cb945fef001b090 (patch) | |
| tree | 4bdc89830d85da133ae48d2a0ad25bd24828ca1d /lib/MC/MCParser | |
| parent | 12649372124260d05afa7f73802202c522f5d376 (diff) | |
| download | external_llvm-cbf8a98c7c652e96967623c80cb945fef001b090.zip external_llvm-cbf8a98c7c652e96967623c80cb945fef001b090.tar.gz external_llvm-cbf8a98c7c652e96967623c80cb945fef001b090.tar.bz2 | |
fix the asmparser so that the target is responsible for skipping to
the end of the line on a parser error, allowing skipping to happen
for syntactic errors but not for semantic errors. Before we would
miss emitting a diagnostic about the second line, because we skipped
it due to the semantic error on the first line:
foo %eax
bar %al
This fixes rdar://8414033 - llvm-mc ignores lines after an invalid instruction mnemonic errors
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@113688 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/MC/MCParser')
| -rw-r--r-- | lib/MC/MCParser/AsmParser.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/MC/MCParser/AsmParser.cpp b/lib/MC/MCParser/AsmParser.cpp index 13aaeba..0a664fd 100644 --- a/lib/MC/MCParser/AsmParser.cpp +++ b/lib/MC/MCParser/AsmParser.cpp @@ -965,7 +965,9 @@ bool AsmParser::ParseStatement() { for (unsigned i = 0, e = ParsedOperands.size(); i != e; ++i) delete ParsedOperands[i]; - return HadError; + // Don't skip the rest of the line, the instruction parser is responsible for + // that. + return false; } MacroInstantiation::MacroInstantiation(const Macro *M, SMLoc IL, SMLoc EL, |