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

Unified Diff: runtime/vm/flow_graph_inliner.cc

Issue 881063003: Cleanups: parsed_function()->function() => function() (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 5 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
« no previous file with comments | « runtime/vm/flow_graph_compiler.cc ('k') | runtime/vm/flow_graph_optimizer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/flow_graph_inliner.cc
===================================================================
--- runtime/vm/flow_graph_inliner.cc (revision 43198)
+++ runtime/vm/flow_graph_inliner.cc (working copy)
@@ -197,7 +197,7 @@
FlowGraph* flow_graph)
: call(call_arg),
ratio(0.0),
- caller(&flow_graph->parsed_function()->function()) {}
+ caller(&flow_graph->function()) {}
};
struct StaticCallInfo {
@@ -207,7 +207,7 @@
StaticCallInfo(StaticCallInstr* value, FlowGraph* flow_graph)
: call(value),
ratio(0.0),
- caller(&flow_graph->parsed_function()->function()) {}
+ caller(&flow_graph->function()) {}
};
struct ClosureCallInfo {
@@ -215,7 +215,7 @@
const Function* caller;
ClosureCallInfo(ClosureCallInstr* value, FlowGraph* flow_graph)
: call(value),
- caller(&flow_graph->parsed_function()->function()) {}
+ caller(&flow_graph->function()) {}
};
const GrowableArray<InstanceCallInfo>& instance_calls() const {
@@ -291,7 +291,7 @@
FlowGraph* graph,
intptr_t depth,
GrowableArray<InlinedInfo>* inlined_info) {
- const Function* caller = &graph->parsed_function()->function();
+ const Function* caller = &graph->function();
Function& target = Function::ZoneHandle();
for (BlockIterator block_it = graph->postorder_iterator();
!block_it.Done();
@@ -358,7 +358,7 @@
// Method not inlined because inlining too deep and method
// not recognized.
if (FLAG_print_inlining_tree) {
- const Function* caller = &graph->parsed_function()->function();
+ const Function* caller = &graph->function();
const Function* target =
&Function::ZoneHandle(
instance_call->ic_data().GetTargetAt(0));
@@ -375,7 +375,7 @@
// Method not inlined because inlining too deep and method
// not recognized.
if (FLAG_print_inlining_tree) {
- const Function* caller = &graph->parsed_function()->function();
+ const Function* caller = &graph->function();
const Function* target = &static_call->function();
inlined_info->Add(InlinedInfo(
caller, target, depth + 1, static_call, "Too deep"));
@@ -534,7 +534,7 @@
void InlineCalls() {
// If inlining depth is less then one abort.
if (FLAG_inlining_depth_threshold < 1) return;
- if (caller_graph_->parsed_function()->function().deoptimization_counter() >=
+ if (caller_graph_->function().deoptimization_counter() >=
FLAG_deoptimization_counter_inlining_threshold) {
return;
}
@@ -851,7 +851,7 @@
caller_graph()->AddToDeferredPrefixes(callee_graph->deferred_prefixes());
FlowGraphInliner::SetInliningId(*callee_graph,
- inliner_->NextInlineId(callee_graph->parsed_function()->function()));
+ inliner_->NextInlineId(callee_graph->function()));
// We allocate a ZoneHandle for the unoptimized code so that it cannot be
// disconnected from its function during the rest of compilation.
Code::ZoneHandle(unoptimized_code.raw());
@@ -1733,7 +1733,7 @@
void FlowGraphInliner::CollectGraphInfo(FlowGraph* flow_graph, bool force) {
- const Function& function = flow_graph->parsed_function()->function();
+ const Function& function = flow_graph->function();
if (force || (function.optimized_instruction_count() == 0)) {
GraphInfoCollector info;
info.Collect(*flow_graph);
@@ -1788,7 +1788,7 @@
// We might later use it for an early bailout from the inlining.
CollectGraphInfo(flow_graph_);
- const Function& top = flow_graph_->parsed_function()->function();
+ const Function& top = flow_graph_->function();
if ((FLAG_inlining_filter != NULL) &&
(strstr(top.ToFullyQualifiedCString(), FLAG_inlining_filter) == NULL)) {
return;
@@ -1799,7 +1799,7 @@
if (FLAG_trace_inlining &&
(FLAG_print_flow_graph || FLAG_print_flow_graph_optimized)) {
OS::Print("Before Inlining of %s\n", flow_graph_->
- parsed_function()->function().ToFullyQualifiedCString());
+ function().ToFullyQualifiedCString());
FlowGraphPrinter printer(*flow_graph_);
printer.PrintBlocks();
}
@@ -1816,7 +1816,7 @@
OS::Print("Inlining growth factor: %f\n", inliner.GrowthFactor());
if (FLAG_print_flow_graph || FLAG_print_flow_graph_optimized) {
OS::Print("After Inlining of %s\n", flow_graph_->
- parsed_function()->function().ToFullyQualifiedCString());
+ function().ToFullyQualifiedCString());
FlowGraphPrinter printer(*flow_graph_);
printer.PrintBlocks();
}
« no previous file with comments | « runtime/vm/flow_graph_compiler.cc ('k') | runtime/vm/flow_graph_optimizer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698