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

Unified Diff: runtime/vm/compiler/backend/inliner.h

Issue 2994283002: [VM-Compiler] Don't inline if we don't have inlining budget enough to fully inline.
Patch Set: Do InliningDecision the way Slava suggested Created 3 years, 3 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/clustered_snapshot.cc ('k') | runtime/vm/compiler/backend/inliner.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/compiler/backend/inliner.h
diff --git a/runtime/vm/compiler/backend/inliner.h b/runtime/vm/compiler/backend/inliner.h
index 4fdd65d1ff29392b811ca0330121ba9c508f69fb..fe243e7fe09a1ff65cdaa987906e86af15d11332 100644
--- a/runtime/vm/compiler/backend/inliner.h
+++ b/runtime/vm/compiler/backend/inliner.h
@@ -32,8 +32,9 @@ class FlowGraphInliner : ValueObject {
GrowableArray<intptr_t>* inlining_black_list,
Precompiler* precompiler);
- // The flow graph is destructively updated upon inlining.
- void Inline();
+ // The flow graph is destructively updated upon inlining. Returns the max
+ // depth that we inlined.
+ int Inline();
// Compute graph info if it was not already computed or if 'force' is true.
static void CollectGraphInfo(FlowGraph* flow_graph, bool force = false);
« no previous file with comments | « runtime/vm/clustered_snapshot.cc ('k') | runtime/vm/compiler/backend/inliner.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698