RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 03:07:44 +00:00
Code Issues Activity Actions

LibJS: Make marking object indexed properties less allocation-heavy

Browse source 
We were building up a vector with all the values in an object's indexed
property storage, and then iterating over the vector to mark values.
Instead of this, simply iterate over the property storage directly. :^)
This commit is contained in:
Andreas Kling 2020-12-08 13:33:01 +01:00
parent 930fae633e
commit 6c4b823cef
2 changed files with 17 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Libraries/LibJS/Runtime/IndexedProperties.h
View file

@ -169,6 +169,20 @@ public:
Vector<ValueAndAttributes> values_unordered() const;
template<typename Callback>
void for_each_value(Callback callback)
{
if (m_storage->is_simple_storage()) {
for (auto& value : static_cast<SimpleIndexedPropertyStorage&>(*m_storage).elements())
callback(value);
} else {
for (auto& element : static_cast<const GenericIndexedPropertyStorage&>(*m_storage).packed_elements())
callback(element.value);
for (auto& element : static_cast<const GenericIndexedPropertyStorage&>(*m_storage).sparse_elements())
callback(element.value.value);
}
}
private:
void switch_to_generic_storage();

5
Libraries/LibJS/Runtime/Object.cpp
View file

@ -818,8 +818,9 @@ void Object::visit_edges(Cell::Visitor& visitor)
for (auto& value : m_storage)
visitor.visit(value);
for (auto& value : m_indexed_properties.values_unordered())
visitor.visit(value.value);
m_indexed_properties.for_each_value([&visitor](auto& value) {
visitor.visit(value);
});
}
bool Object::has_property(const PropertyName& property_name) const