Changeset 142260 in webkit

Timestamp:

Feb 8, 2013 4:45:44 AM (11 years ago)

Author:

akling@apple.com

Message:

JSC: Lower minimum PropertyTable size.
<​http://webkit.org/b/109247>

Reviewed by Darin Adler.

Lower the minimum table size for PropertyTable from 16 to 8.
3.32 MB progression on Membuster3 (a ~13% reduction in memory used by PropertyTables.)

• runtime/PropertyMapHashTable.h:

(PropertyTable):
(JSC::PropertyTable::sizeForCapacity):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• runtime/PropertyMapHashTable.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-02-08  Andreas Kling  <akling@apple.com>
+
+        JSC: Lower minimum PropertyTable size.
+        <http://webkit.org/b/109247>
+
+        Reviewed by Darin Adler.
+
+        Lower the minimum table size for PropertyTable from 16 to 8.
+        3.32 MB progression on Membuster3 (a ~13% reduction in memory used by PropertyTables.)
+
+        * runtime/PropertyMapHashTable.h:
+        (PropertyTable):
+        (JSC::PropertyTable::sizeForCapacity):
+
 2013-02-07  Roger Fong  <roger_fong@apple.com>
 

trunk/Source/JavaScriptCore/runtime/PropertyMapHashTable.h

@@ -235 +235 @@
     OwnPtr< Vector<PropertyOffset> > m_deletedOffsets;
 
-    static const unsigned MinimumTableSize = 16;
+    static const unsigned MinimumTableSize = 8;
     static const unsigned EmptyEntryIndex = 0;
 };
@@ -588 +588 @@
 inline unsigned PropertyTable::sizeForCapacity(unsigned capacity)
 {
-    if (capacity < 8)
+    if (capacity < MinimumTableSize / 2)
         return MinimumTableSize;
     return nextPowerOf2(capacity + 1) * 2;