[llvm] r314791 - [ARM, Asm] Remove dead code causing MSan failure.

Oliver Stannard via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 3 05:28:28 PDT 2017


Author: olista01
Date: Tue Oct  3 05:28:28 2017
New Revision: 314791

URL: http://llvm.org/viewvc/llvm-project?rev=314791&view=rev
Log:
[ARM, Asm] Remove dead code causing MSan failure.

r314779 caused ErrorInfo to be red uninitialised, but also made this code dead,
so it can just be removed.


Modified:
    llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp

Modified: llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp?rev=314791&r1=314790&r2=314791&view=diff
==============================================================================
--- llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp (original)
+++ llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp Tue Oct  3 05:28:28 2017
@@ -9028,13 +9028,6 @@ bool ARMAsmParser::MatchAndEmitInstructi
   MatchResult = MatchInstruction(Operands, Inst, NearMisses, MatchingInlineAsm,
                                  PendConditionalInstruction, Out);
 
-  SMLoc ErrorLoc;
-  if (ErrorInfo < Operands.size()) {
-    ErrorLoc = ((ARMOperand &)*Operands[ErrorInfo]).getStartLoc();
-    if (ErrorLoc == SMLoc())
-      ErrorLoc = IDLoc;
-  }
-
   switch (MatchResult) {
   case Match_Success:
     // Context sensitive operand constraints aren't handled by the matcher,




More information about the llvm-commits mailing list