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

Unified Diff: runtime/vm/clustered_snapshot.cc

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 | « no previous file | runtime/vm/compiler/backend/inliner.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/clustered_snapshot.cc
diff --git a/runtime/vm/clustered_snapshot.cc b/runtime/vm/clustered_snapshot.cc
index ba4b3d99faca98757e5e35ba3a243e4d79be1bb3..4181ce9e4b40d5fabbc3ab552dac32381a799db9 100644
--- a/runtime/vm/clustered_snapshot.cc
+++ b/runtime/vm/clustered_snapshot.cc
@@ -644,6 +644,7 @@ class FunctionDeserializationCluster : public DeserializationCluster {
func->ptr()->deoptimization_counter_ = 0;
func->ptr()->optimized_instruction_count_ = 0;
func->ptr()->optimized_call_site_count_ = 0;
+ func->ptr()->inlining_depth_ = 0;
#endif
}
}
@@ -670,10 +671,10 @@ class FunctionDeserializationCluster : public DeserializationCluster {
code ^= func.CurrentCode();
if (func.HasCode() && !code.IsDisabled()) {
func.SetInstructions(code);
- func.set_was_compiled(true);
+ func.SetWasCompiled(true);
} else {
func.ClearCode();
- func.set_was_compiled(false);
+ func.SetWasCompiled(false);
}
}
} else {
@@ -682,7 +683,7 @@ class FunctionDeserializationCluster : public DeserializationCluster {
func ^= refs.At(i);
func.ClearICDataArray();
func.ClearCode();
- func.set_was_compiled(false);
+ func.SetWasCompiled(false);
}
}
}
« no previous file with comments | « no previous file | runtime/vm/compiler/backend/inliner.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698