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

Unified Diff: runtime/vm/flow_graph.h

Issue 11856010: Change the inlining context from an enum to a class. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 11 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
Index: runtime/vm/flow_graph.h
diff --git a/runtime/vm/flow_graph.h b/runtime/vm/flow_graph.h
index fd3709c9529ce1d52c7784a1ffecbddc04e14727..c3cffb9229536e578f92b87f6badbe35c3dc7ae6 100644
--- a/runtime/vm/flow_graph.h
+++ b/runtime/vm/flow_graph.h
@@ -17,6 +17,7 @@ class FlowGraphBuilder;
class GraphEntryInstr;
class PhiInstr;
class ReturnInstr;
+class ValueInliningContext;
class BlockIterator : public ValueObject {
public:
@@ -98,9 +99,6 @@ class FlowGraph : public ZoneAllocated {
return graph_entry_;
}
- ZoneGrowableArray<ReturnInstr*>* exits() const { return exits_; }
- void set_exits(ZoneGrowableArray<ReturnInstr*>* exits) { exits_ = exits; }
-
intptr_t alloc_ssa_temp_index() { return current_ssa_temp_index_++; }
intptr_t InstructionCount() const;
@@ -117,7 +115,9 @@ class FlowGraph : public ZoneAllocated {
// body blocks for each loop header.
void ComputeLoops(GrowableArray<BlockEntryInstr*>* loop_headers);
- void InlineCall(Definition* call, FlowGraph* callee_graph);
+ void InlineCall(Definition* call,
+ FlowGraph* callee_graph,
+ ValueInliningContext* inlining_context);
void RepairGraphAfterInlining();
// TODO(zerny): Once the SSA is feature complete this should be removed.
@@ -177,7 +177,6 @@ class FlowGraph : public ZoneAllocated {
GrowableArray<BlockEntryInstr*> preorder_;
GrowableArray<BlockEntryInstr*> postorder_;
GrowableArray<BlockEntryInstr*> reverse_postorder_;
- ZoneGrowableArray<ReturnInstr*>* exits_;
bool invalid_dominator_tree_;
};

Powered by Google App Engine
This is Rietveld 408576698