Commit b8ea338d authored by Santiago Aboy Solanes's avatar Santiago Aboy Solanes Committed by Commit Bot

[compiler] Revert to serialization for some classes

Reverted:
 * FixedDoubleArray
 * BigInt
 * HeapNumber
 * Partial work of JSDataView

Bug: v8:7790
Change-Id: I075e1d6d50129771f6208f198911797c6db3b7cb
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2431944
Commit-Queue: Georg Neis <neis@chromium.org>
Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#70187}
parent a8f46352
...@@ -60,16 +60,12 @@ enum class OddballType : uint8_t { ...@@ -60,16 +60,12 @@ enum class OddballType : uint8_t {
#define HEAP_BROKER_NEVER_SERIALIZED_OBJECT_LIST(V) \ #define HEAP_BROKER_NEVER_SERIALIZED_OBJECT_LIST(V) \
/* Subtypes of FixedArray */ \ /* Subtypes of FixedArray */ \
V(ObjectBoilerplateDescription) \ V(ObjectBoilerplateDescription) \
/* Subtypes of FixedArrayBase */ \
V(FixedDoubleArray) \
/* Subtypes of Name */ \ /* Subtypes of Name */ \
V(Symbol) \ V(Symbol) \
/* Subtypes of HeapObject */ \ /* Subtypes of HeapObject */ \
V(AccessorInfo) \ V(AccessorInfo) \
V(ArrayBoilerplateDescription) \ V(ArrayBoilerplateDescription) \
V(BigInt) \
V(Cell) \ V(Cell) \
V(HeapNumber) \
V(TemplateObjectDescription) V(TemplateObjectDescription)
// This list is sorted such that subtypes appear before their supertypes. // This list is sorted such that subtypes appear before their supertypes.
...@@ -93,6 +89,7 @@ enum class OddballType : uint8_t { ...@@ -93,6 +89,7 @@ enum class OddballType : uint8_t {
/* Subtypes of FixedArrayBase */ \ /* Subtypes of FixedArrayBase */ \
V(BytecodeArray) \ V(BytecodeArray) \
V(FixedArray) \ V(FixedArray) \
V(FixedDoubleArray) \
/* Subtypes of Name */ \ /* Subtypes of Name */ \
V(InternalizedString) \ V(InternalizedString) \
V(String) \ V(String) \
...@@ -100,6 +97,7 @@ enum class OddballType : uint8_t { ...@@ -100,6 +97,7 @@ enum class OddballType : uint8_t {
V(JSObject) \ V(JSObject) \
/* Subtypes of HeapObject */ \ /* Subtypes of HeapObject */ \
V(AllocationSite) \ V(AllocationSite) \
V(BigInt) \
V(CallHandlerInfo) \ V(CallHandlerInfo) \
V(Code) \ V(Code) \
V(DescriptorArray) \ V(DescriptorArray) \
...@@ -107,6 +105,7 @@ enum class OddballType : uint8_t { ...@@ -107,6 +105,7 @@ enum class OddballType : uint8_t {
V(FeedbackVector) \ V(FeedbackVector) \
V(FixedArrayBase) \ V(FixedArrayBase) \
V(FunctionTemplateInfo) \ V(FunctionTemplateInfo) \
V(HeapNumber) \
V(JSReceiver) \ V(JSReceiver) \
V(Map) \ V(Map) \
V(Name) \ V(Name) \
......
...@@ -705,7 +705,6 @@ class HeapNumberData : public HeapObjectData { ...@@ -705,7 +705,6 @@ class HeapNumberData : public HeapObjectData {
HeapNumberData(JSHeapBroker* broker, ObjectData** storage, HeapNumberData(JSHeapBroker* broker, ObjectData** storage,
Handle<HeapNumber> object) Handle<HeapNumber> object)
: HeapObjectData(broker, storage, object), value_(object->value()) { : HeapObjectData(broker, storage, object), value_(object->value()) {
DCHECK(!FLAG_turbo_direct_heap_access);
} }
double value() const { return value_; } double value() const { return value_; }
...@@ -1021,7 +1020,6 @@ class BigIntData : public HeapObjectData { ...@@ -1021,7 +1020,6 @@ class BigIntData : public HeapObjectData {
BigIntData(JSHeapBroker* broker, ObjectData** storage, Handle<BigInt> object) BigIntData(JSHeapBroker* broker, ObjectData** storage, Handle<BigInt> object)
: HeapObjectData(broker, storage, object), : HeapObjectData(broker, storage, object),
as_uint64_(object->AsUint64(nullptr)) { as_uint64_(object->AsUint64(nullptr)) {
DCHECK(!FLAG_turbo_direct_heap_access);
} }
uint64_t AsUint64() const { return as_uint64_; } uint64_t AsUint64() const { return as_uint64_; }
...@@ -1575,7 +1573,6 @@ FixedDoubleArrayData::FixedDoubleArrayData(JSHeapBroker* broker, ...@@ -1575,7 +1573,6 @@ FixedDoubleArrayData::FixedDoubleArrayData(JSHeapBroker* broker,
ObjectData** storage, ObjectData** storage,
Handle<FixedDoubleArray> object) Handle<FixedDoubleArray> object)
: FixedArrayBaseData(broker, storage, object), contents_(broker->zone()) { : FixedArrayBaseData(broker, storage, object), contents_(broker->zone()) {
DCHECK(!FLAG_turbo_direct_heap_access);
} }
void FixedDoubleArrayData::SerializeContents(JSHeapBroker* broker) { void FixedDoubleArrayData::SerializeContents(JSHeapBroker* broker) {
...@@ -3361,7 +3358,7 @@ BIMODAL_ACCESSOR(JSBoundFunction, JSReceiver, bound_target_function) ...@@ -3361,7 +3358,7 @@ BIMODAL_ACCESSOR(JSBoundFunction, JSReceiver, bound_target_function)
BIMODAL_ACCESSOR(JSBoundFunction, Object, bound_this) BIMODAL_ACCESSOR(JSBoundFunction, Object, bound_this)
BIMODAL_ACCESSOR(JSBoundFunction, FixedArray, bound_arguments) BIMODAL_ACCESSOR(JSBoundFunction, FixedArray, bound_arguments)
BIMODAL_ACCESSOR_WITH_FLAG_C(JSDataView, size_t, byte_length) BIMODAL_ACCESSOR_C(JSDataView, size_t, byte_length)
BIMODAL_ACCESSOR_C(JSFunction, bool, has_feedback_vector) BIMODAL_ACCESSOR_C(JSFunction, bool, has_feedback_vector)
BIMODAL_ACCESSOR_C(JSFunction, bool, has_initial_map) BIMODAL_ACCESSOR_C(JSFunction, bool, has_initial_map)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment