Always flush pending errors in MCAsmParser

This has become visible with the --fatal-warnings support.

Author: jsonn

llvm/lib/MC/MCParser/AsmParser.cpp

@@ -916,21 +916,20 @@ bool AsmParser::Run(bool NoInitialTextSection, bool NoFinalize) {
   // While we have input, parse each statement.
   while (Lexer.isNot(AsmToken::Eof)) {
     ParseStatementInfo Info(&AsmStrRewrites);
-    if (!parseStatement(Info, nullptr))
-      continue;
+    bool Parsed = parseStatement(Info, nullptr);
 
     // If we have a Lexer Error we are on an Error Token. Load in Lexer Error
     // for printing ErrMsg via Lex() only if no (presumably better) parser error
     // exists.
-    if (!hasPendingError() && Lexer.getTok().is(AsmToken::Error)) {
+    if (Parsed && !hasPendingError() && Lexer.getTok().is(AsmToken::Error)) {
       Lex();
     }
 
     // parseStatement returned true so may need to emit an error.
     printPendingErrors();
 
     // Skipping to the next line if needed.
-    if (!getLexer().isAtStartOfStatement())
+    if (Parsed && !getLexer().isAtStartOfStatement())
       eatToEndOfStatement();
   }
 

llvm/test/MC/ARM/arm-load-store-multiple-deprecated.s

@@ -1,5 +1,7 @@
 @ RUN: llvm-mc -triple armv6t2-linux-eabi -filetype asm -o - %s 2>&1 \
 @ RUN:   | FileCheck %s
+@ RUN: not llvm-mc -triple armv6t2-linux-eabi --fatal-warnings -filetype asm -o - %s 2>&1 \
+@ RUN:   | FileCheck -check-prefix CHECK-ERROR %s
 
 @ RUN: not llvm-mc -triple armv7-linux-eabi -filetype asm -o - %s 2>&1 \
 @ RUN:   | FileCheck %s -check-prefix CHECK -check-prefix CHECK-V7
@@ -12,10 +14,12 @@
 stm:
 	stm sp!, {r0, pc}
 @ CHECK: warning: use of SP or PC in the list is deprecated
+@ CHECK-ERROR: error: use of SP or PC in the list is deprecated
 @ CHECK: stm sp!, {r0, pc}
 @ CHECK: ^
 	stm r0!, {r0, sp}
 @ CHECK: warning: use of SP or PC in the list is deprecated
+@ CHECK-ERROR: error: use of SP or PC in the list is deprecated
 @ CHECK: stm r0!, {r0, sp}
 @ CHECK: ^
 	stm r1!, {r0, sp, pc}