beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-29 08:11:13 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Revert "LibJS: Cache the shape for normal ECMAScriptFunctionObject prototypes"

Browse source 
This reverts commit 749cf2d1b5.

This broke pinterest.com
This commit is contained in:
Andreas Kling 2024-09-28 16:20:29 +02:00
parent 20af2eb2b0
commit 061ad33705
Notes: github-actions[bot] 2024-09-28 17:42:53 +00:00 
Author: https://github.com/awesomekling
Commit: https://github.com/LadybirdBrowser/ladybird/commit/061ad337057
3 changed files with 2 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp
View file

@ -354,8 +354,8 @@ void ECMAScriptFunctionObject::initialize(Realm& realm)
Object* prototype = nullptr; Object* prototype = nullptr;
switch (m_kind) { switch (m_kind) {
case FunctionKind::Normal: case FunctionKind::Normal:
prototype = Object::create_with_premade_shape(realm.intrinsics().new_function_object_prototype_shape()); prototype = Object::create_prototype(realm, realm.intrinsics().object_prototype());
prototype->put_direct(realm.intrinsics().new_function_object_prototype_constructor_offset(), this); MUST(prototype->define_property_or_throw(vm.names.constructor, { .value = this, .writable = true, .enumerable = false, .configurable = true }));
break; break;
case FunctionKind::Generator: case FunctionKind::Generator:
// prototype is "g1.prototype" in figure-2 (https://tc39.es/ecma262/img/figure-2.png) // prototype is "g1.prototype" in figure-2 (https://tc39.es/ecma262/img/figure-2.png)

7
Userland/Libraries/LibJS/Runtime/Intrinsics.cpp
View file

@ -188,12 +188,6 @@ ThrowCompletionOr<void> Intrinsics::initialize_intrinsics(Realm& realm)
m_new_object_shape = heap().allocate_without_realm<Shape>(realm); m_new_object_shape = heap().allocate_without_realm<Shape>(realm);
m_new_object_shape->set_prototype_without_transition(m_object_prototype); m_new_object_shape->set_prototype_without_transition(m_object_prototype);
m_new_function_object_prototype_shape = heap().allocate_without_realm<Shape>(realm);
m_new_function_object_prototype_shape->set_prototype_shape();
m_new_function_object_prototype_shape->set_prototype_without_transition(m_object_prototype);
m_new_function_object_prototype_shape->add_property_without_transition(vm.names.constructor, Attribute::Writable | Attribute::Configurable | Attribute::Enumerable);
m_new_function_object_prototype_constructor_offset = m_new_function_object_prototype_shape->lookup(vm.names.constructor.to_string_or_symbol()).value().offset;
// OPTIMIZATION: A lot of runtime algorithms create an "iterator result" object. // OPTIMIZATION: A lot of runtime algorithms create an "iterator result" object.
// We pre-bake a shape for these objects and remember the property offsets. // We pre-bake a shape for these objects and remember the property offsets.
// This allows us to construct them very quickly. // This allows us to construct them very quickly.
@ -373,7 +367,6 @@ void Intrinsics::visit_edges(Visitor& visitor)
visitor.visit(m_realm); visitor.visit(m_realm);
visitor.visit(m_empty_object_shape); visitor.visit(m_empty_object_shape);
visitor.visit(m_new_object_shape); visitor.visit(m_new_object_shape);
visitor.visit(m_new_function_object_prototype_shape);
visitor.visit(m_iterator_result_object_shape); visitor.visit(m_iterator_result_object_shape);
visitor.visit(m_proxy_constructor); visitor.visit(m_proxy_constructor);
visitor.visit(m_async_from_sync_iterator_prototype); visitor.visit(m_async_from_sync_iterator_prototype);

6
Userland/Libraries/LibJS/Runtime/Intrinsics.h
View file

@ -22,9 +22,6 @@ public:
NonnullGCPtr<Shape> new_object_shape() { return *m_new_object_shape; } NonnullGCPtr<Shape> new_object_shape() { return *m_new_object_shape; }
[[nodiscard]] NonnullGCPtr<Shape> new_function_object_prototype_shape() { return *m_new_function_object_prototype_shape; }
[[nodiscard]] u32 new_function_object_prototype_constructor_offset() { return m_new_function_object_prototype_constructor_offset; }
[[nodiscard]] NonnullGCPtr<Shape> iterator_result_object_shape() { return *m_iterator_result_object_shape; } [[nodiscard]] NonnullGCPtr<Shape> iterator_result_object_shape() { return *m_iterator_result_object_shape; }
[[nodiscard]] u32 iterator_result_object_value_offset() { return m_iterator_result_object_value_offset; } [[nodiscard]] u32 iterator_result_object_value_offset() { return m_iterator_result_object_value_offset; }
[[nodiscard]] u32 iterator_result_object_done_offset() { return m_iterator_result_object_done_offset; } [[nodiscard]] u32 iterator_result_object_done_offset() { return m_iterator_result_object_done_offset; }
@ -128,9 +125,6 @@ private:
GCPtr<Shape> m_empty_object_shape; GCPtr<Shape> m_empty_object_shape;
GCPtr<Shape> m_new_object_shape; GCPtr<Shape> m_new_object_shape;
GCPtr<Shape> m_new_function_object_prototype_shape;
u32 m_new_function_object_prototype_constructor_offset { 0 };
GCPtr<Shape> m_iterator_result_object_shape; GCPtr<Shape> m_iterator_result_object_shape;
u32 m_iterator_result_object_value_offset { 0 }; u32 m_iterator_result_object_value_offset { 0 };
u32 m_iterator_result_object_done_offset { 0 }; u32 m_iterator_result_object_done_offset { 0 };