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

Unified Diff: runtime/vm/object.cc

Issue 1392893003: In precompilation, finalize all classes eagerly. Use the stable class hierarchy for doing CHA based… (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 2 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
Index: runtime/vm/object.cc
diff --git a/runtime/vm/object.cc b/runtime/vm/object.cc
index a092c95cde3220c00e3fd04463f3b4ce3683bd46..d50ff0cc1f074cfee29efcd1bff0e19b10979f5a 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -2773,6 +2773,7 @@ void Class::set_invocation_dispatcher_cache(const Array& cache) const {
void Class::Finalize() const {
+ ASSERT(IsSignatureClass() || !Isolate::Current()->all_classes_finalized());
rmacnak 2015/10/14 16:38:48 Add note that signature classes may be added after
srdjan 2015/10/14 16:51:58 Done.
ASSERT(!is_finalized());
// Prefinalized classes have a VM internal representation and no Dart fields.
// Their instance size is precomputed and field offsets are known.
@@ -3037,7 +3038,6 @@ RawObject* Class::Evaluate(const String& expr,
// Ensure that top level parsing of the class has been done.
-// TODO(24109): Migrate interface to Thread*.
RawError* Class::EnsureIsFinalized(Thread* thread) const {
// Finalized classes have already been parsed.
if (is_finalized()) {
« no previous file with comments | « runtime/vm/isolate.cc ('k') | runtime/vm/precompiler.h » ('j') | runtime/vm/precompiler.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698