Changeset 42606 in webkit for trunk/JavaScriptCore/runtime/Structure.cpp

Timestamp:

Apr 16, 2009, 11:20:32 PM (16 years ago)

Author:

Stephanie Lewis

Message:

2009-04-16 Stephanie Lewis <Stephanie Lewis>

Reviewed by Oliver Hunt.

<rdar://problem/6744652> 32-bit to 64-bit: Javascript hash tables double in size

Remove perfect hash optimization which removes 1 MB of overhead on 32-bit and almost 2 MB on 64-bit. Removing the optimization was not a regression on SunSpider and the acid 3 test still passes.

• create_hash_table:
• runtime/Lookup.cpp: (JSC::HashTable::createTable): (JSC::HashTable::deleteTable):
• runtime/Lookup.h: (JSC::HashEntry::initialize): (JSC::HashEntry::next): (JSC::HashTable::entry):
• runtime/Structure.cpp: (JSC::Structure::getEnumerableNamesFromClassInfoTable):

File:

• trunk/JavaScriptCore/runtime/Structure.cpp (modified) (1 diff)

trunk/JavaScriptCore/runtime/Structure.cpp

@@ -949 +949 @@
         table->initializeIfNeeded(exec);
         ASSERT(table->table);
-#if ENABLE(PERFECT_HASH_SIZE)
-        int hashSizeMask = table->hashSizeMask;
-#else
+
         int hashSizeMask = table->compactSize - 1;
-#endif
         const HashEntry* entry = table->table;
         for (int i = 0; i <= hashSizeMask; ++i, ++entry) {