MIPS: r6 compact branch optimization.

src/ic/mips/ic-mips.cc

Index: src/ic/mips/ic-mips.cc
diff --git a/src/ic/mips/ic-mips.cc b/src/ic/mips/ic-mips.cc
index beff9a90a67b61684c4a0c42ec7b055a9abb2625..7b4289e7f37aeb10b3f082a8dfb16ec0a06618af 100644
--- a/src/ic/mips/ic-mips.cc
+++ b/src/ic/mips/ic-mips.cc
@@ -902,12 +902,7 @@ void PatchInlinedSmiCode(Address address, InlinedSmiCheck check) {
     patcher.masm()->andi(at, reg, 0);
   }
   DCHECK(Assembler::IsBranch(branch_instr));
-  if (Assembler::IsBeq(branch_instr)) {
-    patcher.ChangeBranchCondition(ne);
-  } else {
-    DCHECK(Assembler::IsBne(branch_instr));
-    patcher.ChangeBranchCondition(eq);
-  }
+  patcher.ChangeBranchCondition();

[paul.l..., 2015/10/15 01:51:24]

I kind of like the old way, where you check if (eq) set-to(ne), etc.

What you have now is a just a toggle. Could this get out of sync in any way?

(I do understand extra complexity in the implementation of
ChangeBranchCondition())

[balazs.kilvady, 2015/10/30 21:11:14]

Done.

 }
 }  // namespace internal
 }  // namespace v8