Refactored FlowGraphBuilder into a separate FlowGraph representation.

runtime/vm/flow_graph_allocator.h

Index: runtime/vm/flow_graph_allocator.h
diff --git a/runtime/vm/flow_graph_allocator.h b/runtime/vm/flow_graph_allocator.h
index 25e5fd3ece609c1a0c84cdb7ae0a403eae77cce3..36e9631b2bfc490cc749863e6d0960526e37447f 100644
--- a/runtime/vm/flow_graph_allocator.h
+++ b/runtime/vm/flow_graph_allocator.h
@@ -12,15 +12,14 @@ namespace dart {
 
 class AllocationFinger;
 class BlockInfo;
-class FlowGraphBuilder;
+class FlowGraph;
 class LiveRange;
 class UseInterval;
 class UsePosition;
 
 class FlowGraphAllocator : public ValueObject {
  public:
-  FlowGraphAllocator(const GrowableArray<BlockEntryInstr*>& block_order,
-                     FlowGraphBuilder* builder);
+  explicit FlowGraphAllocator(FlowGraph* flow_graph);
 
   void AllocateRegisters();
 
@@ -176,7 +175,7 @@ class FlowGraphAllocator : public ValueObject {
 
   // TODO(vegorov): this field is used only to call Bailout. Remove when

[Kevin Millikin (Google), 2012/08/16 08:09:57]

I don't think the TODO comment is accurate anymore (flow_graph_ is used for more
than calling Bailout).  The comment can be deleted.

[zerny-google, 2012/08/16 11:52:27]

Done.

   // all bailouts are gone.
-  FlowGraphBuilder* builder_;
+  FlowGraph* flow_graph_;
 
   const GrowableArray<BlockEntryInstr*>& block_order_;
   const GrowableArray<BlockEntryInstr*>& postorder_;