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

Unified Diff: runtime/vm/compiler.cc

Issue 11567012: Store optimized flow graph statistics on the function itself. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years 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.h » ('j') | runtime/vm/flow_graph_inliner.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/compiler.cc
diff --git a/runtime/vm/compiler.cc b/runtime/vm/compiler.cc
index 9ba029666b3adafd52ca85900d0b46a7a0f9e719..7a17c7ddb51fe55ee0bd6cc6cc93ef79b49bc056 100644
--- a/runtime/vm/compiler.cc
+++ b/runtime/vm/compiler.cc
@@ -183,6 +183,10 @@ static bool CompileParsedFunctionHelper(const ParsedFunction& parsed_function,
// Inlining (mutates the flow graph)
if (FLAG_use_inlining) {
+ // Collect graph info and store it on the function.
+ // We might later use it for an early bailout from the inlining.
+ FlowGraphInliner::CollectGraphInfo(flow_graph);
Kevin Millikin (Google) 2012/12/13 16:43:40 Why not move this down into FlowGraphInliner::Inli
Vyacheslav Egorov (Google) 2012/12/13 18:54:46 Done.
+
TimerScope timer(FLAG_compiler_stats,
&CompilerStats::graphinliner_timer);
FlowGraphInliner inliner(flow_graph);
« no previous file with comments | « no previous file | runtime/vm/flow_graph_inliner.h » ('j') | runtime/vm/flow_graph_inliner.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698