Changeset 28777 in webkit for trunk/JavaScriptCore/JavaScriptCore.xcodeproj

Timestamp:

Dec 16, 2007, 2:53:51 PM (17 years ago)

Author:

[email protected]

Message:

Reviewed by Darin Adler and Maciej Stachowiak.

More refactoring to support global variable optimization.

Changed SymbolTable to use RefPtr<UString::Rep> as its key instead of
UString::Rep*. With globals, the symbol table can outlast the
declaration node for any given symbol, so the symbol table needs to ref
its symbol names.

In support, specialized HashMaps with RefPtr keys to allow lookup
via raw pointer, avoiding refcount churn.

SunSpider reports a .6% speedup (prolly just noise).

• JavaScriptCore.vcproj/WTF/WTF.vcproj: Added new file: wtf/RefPtrHashMap.h
• JavaScriptCore.xcodeproj/project.pbxproj: ditto

• kjs/JSVariableObject.cpp: (KJS::JSVariableObject::getPropertyNames): Symbol table keys are RefPtrs now.

• kjs/SymbolTable.h: Modified key traits to match RefPtr. Added a static Rep* for null, which helps compute the deletedValue() trait.

• wtf/HashMap.h: #include the RefPtr specialization so everyone can use it.

• wtf/RefPtrHashMap.h: Copied from wtf/HashMap.h. Added overloaded versions of find(), contains(), get(), set(), add(), remove(), and take() that take raw pointers as keys.

File:

• trunk/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj (modified) (5 diffs)

trunk/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj

@@ -68 +68 @@
                 1483B58A099BC1950016E4F0 /* JSImmediate.h in Headers */ = {isa = PBXBuildFile; fileRef = 1483B589099BC1950016E4F0 /* JSImmediate.h */; settings = {ATTRIBUTES = (Private, ); }; };
                 148A1627095D16BB00666D0D /* ListRefPtr.h in Headers */ = {isa = PBXBuildFile; fileRef = 148A1626095D16BB00666D0D /* ListRefPtr.h */; settings = {ATTRIBUTES = (Private, ); }; };
+                148A1ECE0D10C23B0069A47C /* RefPtrHashMap.h in Headers */ = {isa = PBXBuildFile; fileRef = 148A1ECD0D10C23B0069A47C /* RefPtrHashMap.h */; settings = {ATTRIBUTES = (Private, ); }; };
                 14A396A70CD2933100B5B4FF /* SymbolTable.h in Headers */ = {isa = PBXBuildFile; fileRef = 14A396A60CD2933100B5B4FF /* SymbolTable.h */; settings = {ATTRIBUTES = (Private, ); }; };
                 14ABB36F099C076400E2A24F /* value.h in Headers */ = {isa = PBXBuildFile; fileRef = 14ABB36E099C076400E2A24F /* value.h */; settings = {ATTRIBUTES = (Private, ); }; };
@@ -425 +426 @@
                 1483B589099BC1950016E4F0 /* JSImmediate.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = JSImmediate.h; sourceTree = "<group>"; };
                 148A1626095D16BB00666D0D /* ListRefPtr.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = ListRefPtr.h; sourceTree = "<group>"; };
+                148A1ECD0D10C23B0069A47C /* RefPtrHashMap.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = RefPtrHashMap.h; sourceTree = "<group>"; };
                 14A396A60CD2933100B5B4FF /* SymbolTable.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; path = SymbolTable.h; sourceTree = "<group>"; };
                 14ABB36E099C076400E2A24F /* value.h */ = {isa = PBXFileReference; fileEncoding = 30; lastKnownFileType = sourcecode.c.h; path = value.h; sourceTree = "<group>"; };
@@ -908 +910 @@
                         isa = PBXGroup;
                         children = (
-                                E195678D09E7CF1200B89D13 /* unicode */,
                                 93AA4F770957251F0084B3A7 /* AlwaysInline.h */,
                                 938C4F690CA06BC700D9310A /* ASCIICType.h */,
@@ -938 +939 @@
                                 1419D32C0CEA7CDE00FF507A /* RefCounted.h */,
                                 65C647B3093EF8D60022C380 /* RefPtr.h */,
+                                148A1ECD0D10C23B0069A47C /* RefPtrHashMap.h */,
                                 51F648D60BB4E2CA0033D760 /* RetainPtr.h */,
                                 E11D51750B2E798D0056C188 /* StringExtras.h */,
+                                5DA479650CFBCF56009328A0 /* TCPackedCache.h */,
                                 6541BD6E08E80A17002CBEE7 /* TCPageMap.h */,
                                 6541BD6F08E80A17002CBEE7 /* TCSpinLock.h */,
-                                5DA479650CFBCF56009328A0 /* TCPackedCache.h */,
                                 6541BD7008E80A17002CBEE7 /* TCSystemAlloc.cpp */,
                                 6541BD7108E80A17002CBEE7 /* TCSystemAlloc.h */,
+                                E195678D09E7CF1200B89D13 /* unicode */,
                                 935AF46B09E9D9DB00ACD1D8 /* UnusedParam.h */,
                                 6592C316098B7DE10003D4F6 /* Vector.h */,
@@ -1276 +1279 @@
                                 5DA479660CFBCF56009328A0 /* TCPackedCache.h in Headers */,
                                 14F252570D08DD8D004ECFFF /* JSVariableObject.h in Headers */,
+                                148A1ECE0D10C23B0069A47C /* RefPtrHashMap.h in Headers */,
                         );
                         runOnlyForDeploymentPostprocessing = 0;