Title: [142260] trunk/Source/_javascript_Core
Revision
142260
Author
[email protected]
Date
2013-02-08 04:45:44 -0800 (Fri, 08 Feb 2013)

Log 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):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (142259 => 142260)


--- trunk/Source/_javascript_Core/ChangeLog	2013-02-08 12:42:31 UTC (rev 142259)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-02-08 12:45:44 UTC (rev 142260)
@@ -1,3 +1,17 @@
+2013-02-08  Andreas Kling  <[email protected]>
+
+        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  <[email protected]>
 
         Unreviewed. More VS2010 WebKit solution touchups.

Modified: trunk/Source/_javascript_Core/runtime/PropertyMapHashTable.h (142259 => 142260)


--- trunk/Source/_javascript_Core/runtime/PropertyMapHashTable.h	2013-02-08 12:42:31 UTC (rev 142259)
+++ trunk/Source/_javascript_Core/runtime/PropertyMapHashTable.h	2013-02-08 12:45:44 UTC (rev 142260)
@@ -234,7 +234,7 @@
     unsigned m_deletedCount;
     OwnPtr< Vector<PropertyOffset> > m_deletedOffsets;
 
-    static const unsigned MinimumTableSize = 16;
+    static const unsigned MinimumTableSize = 8;
     static const unsigned EmptyEntryIndex = 0;
 };
 
@@ -587,7 +587,7 @@
 
 inline unsigned PropertyTable::sizeForCapacity(unsigned capacity)
 {
-    if (capacity < 8)
+    if (capacity < MinimumTableSize / 2)
         return MinimumTableSize;
     return nextPowerOf2(capacity + 1) * 2;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to