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

Unified Diff: runtime/vm/class_finalizer.cc

Issue 2901533002: [kernel] Stream everything. Replace .kernel_function with .kernel_offset (Closed)
Patch Set: Fixed assert issues; small refactorings. Created 3 years, 7 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/class_finalizer.cc
diff --git a/runtime/vm/class_finalizer.cc b/runtime/vm/class_finalizer.cc
index 288e05698389fed9323d941b3c02cae9f5e03dc9..13ac6751d3612cac1c4d2e1a3ea87e1228f3a7be 100644
--- a/runtime/vm/class_finalizer.cc
+++ b/runtime/vm/class_finalizer.cc
@@ -1631,7 +1631,7 @@ void ClassFinalizer::ResolveAndFinalizeMemberTypes(const Class& cls) {
/* is_native = */ false, cls, field.token_pos()));
getter.set_result_type(type);
getter.set_is_debuggable(false);
- getter.set_kernel_function(field.kernel_field());
+ getter.set_kernel_offset(field.kernel_offset());
cls.AddFunction(getter);
field.SetStaticValue(Object::sentinel(), true);
}
@@ -2389,7 +2389,7 @@ void ClassFinalizer::ApplyMixinMembers(const Class& cls) {
} else {
for (intptr_t i = 0; i < functions.Length(); i++) {
func ^= functions.At(i);
- ASSERT(func.kernel_function() != 0);
+ ASSERT(func.kernel_offset() > 0);
cloned_funcs.Add(func);
}
}

Powered by Google App Engine
This is Rietveld 408576698