Fix optimization of %: replace IC call even if not optimizable.

runtime/vm/flow_graph_optimizer.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/flow_graph_optimizer.h"
 
 #include "vm/bit_vector.h"
 #include "vm/cha.h"
 #include "vm/flow_graph_builder.h"
 #include "vm/flow_graph_compiler.h"
@@ skipping 635 matching lines @@
       InsertBefore(call,
                    new CheckSmiInstr(left->Copy(), call->deopt_id()),
                    call->env(),
                    Definition::kEffect);
       ConstantInstr* c = new ConstantInstr(Smi::Handle(Smi::New(value - 1)));
       InsertBefore(call, c, NULL, Definition::kValue);
       BinarySmiOpInstr* bin_op =
           new BinarySmiOpInstr(Token::kBIT_AND, call, left, new Value(c));
       call->ReplaceWith(bin_op, current_iterator());
       RemovePushArguments(call);
+    } else {
+      // Did not replace.
+      return false;
     }
   } else {
     ASSERT(operands_type == kSmiCid);
     Value* left = call->ArgumentAt(0)->value();
     Value* right = call->ArgumentAt(1)->value();
     // Insert two smi checks and attach a copy of the original
     // environment because the smi operation can still deoptimize.
     InsertBefore(call,
                  new CheckSmiInstr(left->Copy(), call->deopt_id()),
                  call->env(),
@@ skipping 2669 matching lines @@
 
   if (FLAG_trace_constant_propagation) {
     OS::Print("\n==== After constant propagation ====\n");
     FlowGraphPrinter printer(*graph_);
     printer.PrintBlocks();
   }
 }
 
 
 }  // namespace dart