Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(76)

Unified Diff: runtime/vm/flow_graph_allocator.h

Issue 10857016: Refactored FlowGraphBuilder into a separate FlowGraph representation. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Revision based on Kevin's review. Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/flow_graph.cc ('k') | runtime/vm/flow_graph_allocator.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..ad20fd3a1f085d7e8da3be88aaf6207053ed19a5 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(const FlowGraph& flow_graph);
void AllocateRegisters();
@@ -174,10 +173,7 @@ class FlowGraphAllocator : public ValueObject {
void PrintLiveRanges();
- // TODO(vegorov): this field is used only to call Bailout. Remove when
- // all bailouts are gone.
- FlowGraphBuilder* builder_;
-
+ const FlowGraph& flow_graph_;
const GrowableArray<BlockEntryInstr*>& block_order_;
const GrowableArray<BlockEntryInstr*>& postorder_;
« no previous file with comments | « runtime/vm/flow_graph.cc ('k') | runtime/vm/flow_graph_allocator.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698