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

Unified Diff: runtime/vm/object.cc

Issue 2941483003: Revert "[kernel] Stream everything. Replace .kernel_function with .kernel_offset" (Closed)
Patch Set: Created 3 years, 6 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/object.h ('k') | runtime/vm/parser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/object.cc
diff --git a/runtime/vm/object.cc b/runtime/vm/object.cc
index a1f636e4d98b5df6f09369fbf33c84a111eaf642..62844ac892779b7ad81664024b61fa86d0e9c57b 100644
--- a/runtime/vm/object.cc
+++ b/runtime/vm/object.cc
@@ -2740,7 +2740,7 @@ RawFunction* Function::CreateMethodExtractor(const String& getter_name) const {
extractor.set_parameter_types(Object::extractor_parameter_types());
extractor.set_parameter_names(Object::extractor_parameter_names());
extractor.set_result_type(Object::dynamic_type());
- extractor.set_kernel_offset(kernel_offset());
+ extractor.set_kernel_function(kernel_function());
extractor.set_extracted_method_closure(closure_function);
extractor.set_is_debuggable(false);
@@ -6814,7 +6814,7 @@ RawFunction* Function::New(const String& name,
NOT_IN_PRECOMPILED(result.set_deoptimization_counter(0));
NOT_IN_PRECOMPILED(result.set_optimized_instruction_count(0));
NOT_IN_PRECOMPILED(result.set_optimized_call_site_count(0));
- result.set_kernel_offset(0);
+ result.set_kernel_function(NULL);
result.set_is_optimizable(is_native ? false : true);
result.set_is_inlinable(true);
result.set_allows_hoisting_check_class(true);
@@ -6854,7 +6854,7 @@ RawFunction* Function::Clone(const Class& new_owner) const {
clone.set_deoptimization_counter(0);
clone.set_optimized_instruction_count(0);
clone.set_optimized_call_site_count(0);
- clone.set_kernel_offset(kernel_offset());
+ clone.set_kernel_function(kernel_function());
if (new_owner.NumTypeParameters() > 0) {
// Adjust uninstantiated types to refer to type parameters of the new owner.
const TypeArguments& type_params =
@@ -7010,7 +7010,7 @@ RawFunction* Function::ImplicitClosureFunction() const {
param_name = ParameterNameAt(has_receiver - kClosure + i);
closure_function.SetParameterNameAt(i, param_name);
}
- closure_function.set_kernel_offset(kernel_offset());
+ closure_function.set_kernel_function(kernel_function());
const Type& signature_type =
Type::Handle(zone, closure_function.SignatureType());
@@ -7480,7 +7480,7 @@ void Function::SetDeoptReasonForAll(intptr_t deopt_id,
bool Function::CheckSourceFingerprint(const char* prefix, int32_t fp) const {
- if ((kernel_offset() <= 0) && (SourceFingerprint() != fp)) {
+ if ((kernel_function() == NULL) && (SourceFingerprint() != fp)) {
const bool recalculatingFingerprints = false;
if (recalculatingFingerprints) {
// This output can be copied into a file, then used with sed
@@ -7864,7 +7864,7 @@ void Field::InitializeNew(const Field& result,
result.set_token_pos(token_pos);
result.set_has_initializer(false);
result.set_is_unboxing_candidate(true);
- result.set_kernel_offset(0);
+ result.set_kernel_field(NULL);
Isolate* isolate = Isolate::Current();
// Use field guards if they are enabled and the isolate has never reloaded.
@@ -7944,7 +7944,7 @@ RawField* Field::Clone(const Field& original) const {
Field& clone = Field::Handle();
clone ^= Object::Clone(*this, Heap::kOld);
clone.SetOriginal(original);
- clone.set_kernel_offset(original.kernel_offset());
+ clone.set_kernel_field(original.kernel_field());
return clone.raw();
}
@@ -9121,16 +9121,6 @@ void Script::set_compile_time_constants(const Array& value) const {
}
-void Script::set_kernel_data(const uint8_t* kernel_data) const {
- StoreNonPointer(&raw_ptr()->kernel_data_, kernel_data);
-}
-
-
-void Script::set_kernel_data_size(const intptr_t kernel_data_size) const {
- StoreNonPointer(&raw_ptr()->kernel_data_size_, kernel_data_size);
-}
-
-
void Script::set_kernel_string_offsets(const TypedData& offsets) const {
StorePointer(&raw_ptr()->kernel_string_offsets_, offsets.raw());
}
@@ -10095,7 +10085,7 @@ static RawString* MakeTypeParameterMetaName(Thread* thread,
void Library::AddMetadata(const Object& owner,
const String& name,
TokenPosition token_pos,
- intptr_t kernel_offset) const {
+ kernel::TreeNode* kernel_node) const {
Thread* thread = Thread::Current();
ASSERT(thread->IsMutatorThread());
Zone* zone = thread->zone();
@@ -10108,7 +10098,7 @@ void Library::AddMetadata(const Object& owner,
field.SetFieldType(Object::dynamic_type());
field.set_is_reflectable(false);
field.SetStaticValue(Array::empty_array(), true);
- field.set_kernel_offset(kernel_offset);
+ field.set_kernel_field(kernel_node);
GrowableObjectArray& metadata =
GrowableObjectArray::Handle(zone, this->metadata());
metadata.Add(field, Heap::kOld);
@@ -10118,36 +10108,36 @@ void Library::AddMetadata(const Object& owner,
void Library::AddClassMetadata(const Class& cls,
const Object& tl_owner,
TokenPosition token_pos,
- intptr_t kernel_offset) const {
+ kernel::TreeNode* kernel_node) const {
Thread* thread = Thread::Current();
Zone* zone = thread->zone();
// We use the toplevel class as the owner of a class's metadata field because
// a class's metadata is in scope of the library, not the class.
AddMetadata(tl_owner,
String::Handle(zone, MakeClassMetaName(thread, zone, cls)),
- token_pos, kernel_offset);
+ token_pos, kernel_node);
}
void Library::AddFieldMetadata(const Field& field,
TokenPosition token_pos,
- intptr_t kernel_offset) const {
+ kernel::TreeNode* kernel_node) const {
Thread* thread = Thread::Current();
Zone* zone = thread->zone();
AddMetadata(Object::Handle(zone, field.RawOwner()),
String::Handle(zone, MakeFieldMetaName(thread, zone, field)),
- token_pos, kernel_offset);
+ token_pos, kernel_node);
}
void Library::AddFunctionMetadata(const Function& func,
TokenPosition token_pos,
- intptr_t kernel_offset) const {
+ kernel::TreeNode* kernel_node) const {
Thread* thread = Thread::Current();
Zone* zone = thread->zone();
AddMetadata(Object::Handle(zone, func.RawOwner()),
String::Handle(zone, MakeFunctionMetaName(thread, zone, func)),
- token_pos, kernel_offset);
+ token_pos, kernel_node);
}
@@ -10223,7 +10213,7 @@ RawObject* Library::GetMetadata(const Object& obj) const {
Object& metadata = Object::Handle();
metadata = field.StaticValue();
if (field.StaticValue() == Object::empty_array().raw()) {
- if (field.kernel_offset() > 0) {
+ if (field.kernel_field() != NULL) {
metadata = kernel::EvaluateMetadata(field);
} else {
metadata = Parser::ParseMetadata(field);
« no previous file with comments | « runtime/vm/object.h ('k') | runtime/vm/parser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698