[llvm] r333404 - [mips] Cleanup the code to reduce diff with the upcoming patches. NFC

Simon Atanasyan via llvm-commits llvm-commits at lists.llvm.org
Tue May 29 02:51:33 PDT 2018


Author: atanasyan
Date: Tue May 29 02:51:33 2018
New Revision: 333404

URL: http://llvm.org/viewvc/llvm-project?rev=333404&view=rev
Log:
[mips] Cleanup the code to reduce diff with the upcoming patches. NFC

Modified:
    llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp

Modified: llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp?rev=333404&r1=333403&r2=333404&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp (original)
+++ llvm/trunk/lib/Target/Mips/AsmParser/MipsAsmParser.cpp Tue May 29 02:51:33 2018
@@ -5971,13 +5971,12 @@ MipsAsmParser::parseMemOperand(OperandVe
 bool MipsAsmParser::searchSymbolAlias(OperandVector &Operands) {
   MCAsmParser &Parser = getParser();
   MCSymbol *Sym = getContext().lookupSymbol(Parser.getTok().getIdentifier());
-  if (Sym) {
-    SMLoc S = Parser.getTok().getLoc();
-    const MCExpr *Expr;
-    if (Sym->isVariable())
-      Expr = Sym->getVariableValue();
-    else
-      return false;
+  if (!Sym)
+    return false;
+
+  SMLoc S = Parser.getTok().getLoc();
+  if (Sym->isVariable()) {
+    const MCExpr *Expr = Sym->getVariableValue();
     if (Expr->getKind() == MCExpr::SymbolRef) {
       const MCSymbolRefExpr *Ref = static_cast<const MCSymbolRefExpr *>(Expr);
       StringRef DefSymbol = Ref->getSymbol().getName();
@@ -5987,12 +5986,13 @@ bool MipsAsmParser::searchSymbolAlias(Op
         if (ResTy == MatchOperand_Success) {
           Parser.Lex();
           return true;
-        } else if (ResTy == MatchOperand_ParseFail)
+        }
+        if (ResTy == MatchOperand_ParseFail)
           llvm_unreachable("Should never ParseFail");
-        return false;
       }
     }
   }
+
   return false;
 }
 
@@ -6085,7 +6085,7 @@ MipsAsmParser::matchAnyRegisterWithoutDo
     return MatchOperand_Success;
   }
 
-  LLVM_DEBUG(dbgs() << Parser.getTok().getKind() << "\n");
+  LLVM_DEBUG(dbgs() << Token.getKind() << "\n");
 
   return MatchOperand_NoMatch;
 }




More information about the llvm-commits mailing list