Fuse comparisons that are used by branches together.

runtime/vm/flow_graph_compiler_ia32.h

Index: runtime/vm/flow_graph_compiler_ia32.h
diff --git a/runtime/vm/flow_graph_compiler_ia32.h b/runtime/vm/flow_graph_compiler_ia32.h
index ea4e0aa4b98aa7eb8ee8079194c408205087583e..cef1fa707177c2cd1fe4ef52de86ddd8658ffe9e 100644
--- a/runtime/vm/flow_graph_compiler_ia32.h
+++ b/runtime/vm/flow_graph_compiler_ia32.h
@@ -176,6 +176,9 @@ class FlowGraphCompiler : public ValueObject {
 
   static const int kLocalsOffsetFromFP = (-1 * kWordSize);
 
+  const Bool& true_value() const { return true_value_; }
+  const Bool& false_value() const { return false_value_; }
+
  private:
   friend class DeoptimizationStub;
 
@@ -263,6 +266,9 @@ class FlowGraphCompiler : public ValueObject {
   GrowableArray<DeoptimizationStub*> deopt_stubs_;
   const bool is_optimizing_;
 
+  const Bool& true_value_;
+  const Bool& false_value_;

[srdjan, 2012/06/13 15:29:35]

I find the name 'true_value' ambiguous as it does no specify what bool value it
is (C, Dart, some enum). Better name are, IMO, true_object, bool_true,
true_handle. The C++ code uses Bool::True() to get the RawObject.

[Vyacheslav Egorov (Google), 2012/06/13 16:23:43]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(FlowGraphCompiler);
 };