[turbofan] Do not produce 2 ops for Int32AddWithOverflow.

src/compiler/ia32/instruction-selector-ia32.cc

Index: src/compiler/ia32/instruction-selector-ia32.cc
diff --git a/src/compiler/ia32/instruction-selector-ia32.cc b/src/compiler/ia32/instruction-selector-ia32.cc
index fcb94bd963b4b72068d18d7cec29a2fb21c7eb27..b71c16736071f14d235dd95e801966630f538968 100644
--- a/src/compiler/ia32/instruction-selector-ia32.cc
+++ b/src/compiler/ia32/instruction-selector-ia32.cc
@@ -1329,9 +1329,10 @@ void InstructionSelector::VisitInt32AddWithOverflow(Node* node) {
   if (Node* ovf = NodeProperties::FindProjection(node, 1)) {
     FlagsContinuation cont = FlagsContinuation::ForSet(kOverflow, ovf);
     return VisitBinop(this, node, kIA32Add, &cont);
+  } else {

[Benedikt Meurer, 2016/03/03 17:48:08]

This doesn't seem to be necessary, since there's a return above. Please keep
this version with the return for consistency with the rest of the file.

[Mircea Trofin, 2016/03/03 21:50:13]

Ugh... true. Haven't noticed the return.

+    FlagsContinuation cont;
+    VisitBinop(this, node, kIA32Add, &cont);
   }
-  FlagsContinuation cont;
-  VisitBinop(this, node, kIA32Add, &cont);
 }