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

Unified Diff: runtime/vm/precompiler.cc

Issue 1459443002: VM: Add dart_precompiled build target, a standalone VM without the JIT compiler. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years, 1 month 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
Index: runtime/vm/precompiler.cc
diff --git a/runtime/vm/precompiler.cc b/runtime/vm/precompiler.cc
index c1bf705fe8fed0204a7d713793612f9cec60b8b7..78f8de2411f3d12c430bb9a6eb5a440d866ebe58 100644
--- a/runtime/vm/precompiler.cc
+++ b/runtime/vm/precompiler.cc
@@ -588,6 +588,7 @@ void Precompiler::AddInstantiatedClass(const Class& cls) {
class_count_++;
cls.set_is_allocated(true);
+ cls.EnsureIsFinalized(T);
rmacnak 2015/11/17 20:51:18 All the classes should have been finalized before
Florian Schneider 2015/11/18 11:44:09 Not true for certain signature classes: If you set
rmacnak 2015/11/18 19:00:07 Oh, right. Signature classes may be created as we
changed_ = true;
if (FLAG_trace_precompiler) {

Powered by Google App Engine
This is Rietveld 408576698