[llvm] r187371 - [mips] Add comment and simplify function.

Akira Hatanaka ahatanaka at mips.com
Mon Jul 29 12:08:35 PDT 2013


Author: ahatanak
Date: Mon Jul 29 14:08:34 2013
New Revision: 187371

URL: http://llvm.org/viewvc/llvm-project?rev=187371&view=rev
Log:
[mips] Add comment and simplify function.


Modified:
    llvm/trunk/lib/Target/Mips/InstPrinter/MipsInstPrinter.cpp

Modified: llvm/trunk/lib/Target/Mips/InstPrinter/MipsInstPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mips/InstPrinter/MipsInstPrinter.cpp?rev=187371&r1=187370&r2=187371&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mips/InstPrinter/MipsInstPrinter.cpp (original)
+++ llvm/trunk/lib/Target/Mips/InstPrinter/MipsInstPrinter.cpp Mon Jul 29 14:08:34 2013
@@ -230,35 +230,26 @@ bool MipsInstPrinter::printAlias(const c
 bool MipsInstPrinter::printAlias(const MCInst &MI, raw_ostream &OS) {
   switch (MI.getOpcode()) {
   case Mips::BEQ:
-    if (isReg<Mips::ZERO>(MI, 1) && printAlias("beqz", MI, 0, 2, OS))
-      return true;
-    break;
+    // beq $r0, $zero, $L2 => beqz $r0, $L2
+    return isReg<Mips::ZERO>(MI, 1) && printAlias("beqz", MI, 0, 2, OS);
   case Mips::BEQ64:
-    if (isReg<Mips::ZERO_64>(MI, 1) && printAlias("beqz", MI, 0, 2, OS))
-      return true;
-    break;
+    // beq $r0, $zero, $L2 => beqz $r0, $L2
+    return isReg<Mips::ZERO_64>(MI, 1) && printAlias("beqz", MI, 0, 2, OS);
   case Mips::BNE:
-    if (isReg<Mips::ZERO>(MI, 1) && printAlias("bnez", MI, 0, 2, OS))
-      return true;
-    break;
+    // bne $r0, $zero, $L2 => bnez $r0, $L2
+    return isReg<Mips::ZERO>(MI, 1) && printAlias("bnez", MI, 0, 2, OS);
   case Mips::BNE64:
-    if (isReg<Mips::ZERO_64>(MI, 1) && printAlias("bnez", MI, 0, 2, OS))
-      return true;
-    break;
+    // bne $r0, $zero, $L2 => bnez $r0, $L2
+    return isReg<Mips::ZERO_64>(MI, 1) && printAlias("bnez", MI, 0, 2, OS);
   case Mips::BC1T:
-    if (isReg<Mips::FCC0>(MI, 0) && printAlias("bc1t", MI, 1, OS))
-      return true;
-    break;
+    // bc1t $fcc0, $L1 => bc1t $L1
+    return isReg<Mips::FCC0>(MI, 0) && printAlias("bc1t", MI, 1, OS);
   case Mips::BC1F:
-    if (isReg<Mips::FCC0>(MI, 0) && printAlias("bc1f", MI, 1, OS))
-      return true;
-    break;
+    // bc1f $fcc0, $L1 => bc1f $L1
+    return isReg<Mips::FCC0>(MI, 0) && printAlias("bc1f", MI, 1, OS);
   case Mips::OR:
-    if (isReg<Mips::ZERO>(MI, 2) && printAlias("move", MI, 0, 1, OS))
-      return true;
-    break;
+    // or $r0, $r1, $zero => move $r0, $r1
+    return isReg<Mips::ZERO>(MI, 2) && printAlias("move", MI, 0, 1, OS);
   default: return false;
   }
-
-  return false;
 }





More information about the llvm-commits mailing list