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

Unified Diff: runtime/vm/flow_graph_inliner.cc

Issue 11312200: Eradicate CallSiteInliner::next_ssa_temp_index_. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 1 month 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_inliner.cc
diff --git a/runtime/vm/flow_graph_inliner.cc b/runtime/vm/flow_graph_inliner.cc
index d26fd2d30e1cd27af5e33ac20b5ef9ecc0a52146..5b39c10d9ae4adba61a37ee3057d88a0224bd4d1 100644
--- a/runtime/vm/flow_graph_inliner.cc
+++ b/runtime/vm/flow_graph_inliner.cc
@@ -262,7 +262,6 @@ class CallSiteInliner : public ValueObject {
public:
explicit CallSiteInliner(FlowGraph* flow_graph)
: caller_graph_(flow_graph),
- next_ssa_temp_index_(flow_graph->max_virtual_register_number()),
inlined_(false),
initial_size_(flow_graph->InstructionCount()),
inlined_size_(0),
@@ -451,7 +450,8 @@ class CallSiteInliner : public ValueObject {
&CompilerStats::graphinliner_ssa_timer,
isolate);
// Compute SSA on the callee graph, catching bailouts.
- callee_graph->ComputeSSA(next_ssa_temp_index_, &param_stubs);
+ callee_graph->ComputeSSA(caller_graph_->max_virtual_register_number(),
+ &param_stubs);
callee_graph->ComputeUseLists();
}
@@ -520,7 +520,6 @@ class CallSiteInliner : public ValueObject {
// Plug result in the caller graph.
caller_graph_->InlineCall(call, callee_graph);
- next_ssa_temp_index_ = caller_graph_->max_virtual_register_number();
// Remove push arguments of the call.
for (intptr_t i = 0; i < call->ArgumentCount(); ++i) {
@@ -757,7 +756,6 @@ class CallSiteInliner : public ValueObject {
FlowGraph* caller_graph_;
- intptr_t next_ssa_temp_index_;
bool inlined_;
intptr_t initial_size_;
intptr_t inlined_size_;

Powered by Google App Engine
This is Rietveld 408576698