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

Unified Diff: runtime/vm/flow_graph_builder.cc

Issue 17646002: Use call counts to determine which static calls to inline. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 years, 6 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 | « no previous file | runtime/vm/flow_graph_inliner.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/flow_graph_builder.cc
===================================================================
--- runtime/vm/flow_graph_builder.cc (revision 24433)
+++ runtime/vm/flow_graph_builder.cc (working copy)
@@ -2114,7 +2114,8 @@
new StaticCallInstr(node->token_pos(),
node->function(),
node->arguments()->names(),
- arguments);
+ arguments,
+ owner()->ic_data_array());
if (node->function().is_native()) {
const intptr_t result_cid = GetResultCidOfNative(node->function());
call->set_result_cid(result_cid);
@@ -2224,7 +2225,8 @@
Do(new StaticCallInstr(node->token_pos(),
node->constructor(),
node->arguments()->names(),
- arguments));
+ arguments,
+ owner()->ic_data_array()));
}
@@ -2292,7 +2294,8 @@
new StaticCallInstr(node->token_pos(),
node->constructor(),
node->arguments()->names(),
- arguments);
+ arguments,
+ owner()->ic_data_array());
const intptr_t result_cid = GetResultCidOfListFactory(node);
if (result_cid != kDynamicCid) {
call->set_result_cid(result_cid);
@@ -2647,7 +2650,8 @@
StaticCallInstr* call = new StaticCallInstr(node->token_pos(),
getter_function,
Object::null_array(), // No names
- arguments);
+ arguments,
+ owner()->ic_data_array());
ReturnDefinition(call);
}
@@ -2714,7 +2718,8 @@
call = new StaticCallInstr(node->token_pos(),
setter_function,
Object::null_array(), // No names.
- arguments);
+ arguments,
+ owner()->ic_data_array());
}
if (result_is_needed) {
Do(call);
@@ -2935,7 +2940,8 @@
StaticCallInstr* load = new StaticCallInstr(node->token_pos(),
*super_function,
Object::null_array(),
- arguments);
+ arguments,
+ owner()->ic_data_array());
ReturnDefinition(load);
} else {
// Generate dynamic call to index operator.
@@ -3014,7 +3020,8 @@
new StaticCallInstr(node->token_pos(),
*super_function,
Object::null_array(),
- arguments);
+ arguments,
+ owner()->ic_data_array());
if (result_is_needed) {
Do(store);
return BuildLoadExprTemp();
@@ -3326,7 +3333,8 @@
return new StaticCallInstr(args_pos,
no_such_method_func,
Object::null_array(),
- push_arguments);
+ push_arguments,
+ owner()->ic_data_array());
}
StaticCallInstr* EffectGraphVisitor::BuildThrowNoSuchMethodError(
intptr_t token_pos,
@@ -3382,7 +3390,8 @@
return new StaticCallInstr(token_pos,
func,
Object::null_array(), // No names.
- arguments);
+ arguments,
+ owner()->ic_data_array());
}
« no previous file with comments | « no previous file | runtime/vm/flow_graph_inliner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698