Remove LiteralIdentifierTable
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Jun 2013 03:43:03 +0000 (03:43 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Jun 2013 03:43:03 +0000 (03:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=117613

Reviewed by Geoffrey Garen.

Removed LiteralIdentifierTable since it doesn't seem to have any perf. impact now.

Source/JavaScriptCore:

* runtime/Identifier.cpp:
(JSC::Identifier::add):

Source/WTF:

* wtf/WTFThreadData.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@151578 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/Identifier.cpp
Source/WTF/ChangeLog
Source/WTF/wtf/WTFThreadData.h

index 3eac9a19eda73330f8bc1c77deca9d113878abe8..16e68e51938d358fc91227435aee66406f63d9d9 100644 (file)
@@ -1,3 +1,15 @@
+2013-06-13  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove LiteralIdentifierTable
+        https://bugs.webkit.org/show_bug.cgi?id=117613
+
+        Reviewed by Geoffrey Garen.
+
+        Removed LiteralIdentifierTable since it doesn't seem to have any perf. impact now.
+
+        * runtime/Identifier.cpp:
+        (JSC::Identifier::add):
+
 2013-06-12  Conrad Shultz  <conrad_shultz@apple.com>
 
         JSExport header documentation substitutes "semicolon" for "colon"
index 6e0ad2aea27dd7ee6b8faf7a9b8fbd1d8e18d803..ad61b493a786a48960282adbc1c6bdca4af20bf6 100644 (file)
@@ -96,11 +96,6 @@ PassRefPtr<StringImpl> Identifier::add(VM* vm, const char* c)
         return add(vm, vm->smallStrings.singleCharacterStringRep(c[0]));
 
     IdentifierTable& identifierTable = *vm->identifierTable;
-    LiteralIdentifierTable& literalIdentifierTable = identifierTable.literalTable();
-
-    const LiteralIdentifierTable::iterator& iter = literalIdentifierTable.find(c);
-    if (iter != literalIdentifierTable.end())
-        return iter->value;
 
     HashSet<StringImpl*>::AddResult addResult = identifierTable.add<const LChar*, IdentifierASCIIStringTranslator>(reinterpret_cast<const LChar*>(c));
 
@@ -108,8 +103,6 @@ PassRefPtr<StringImpl> Identifier::add(VM* vm, const char* c)
     // The boolean in the pair tells us if that is so.
     RefPtr<StringImpl> addedString = addResult.isNewEntry ? adoptRef(*addResult.iterator) : *addResult.iterator;
 
-    literalIdentifierTable.add(c, addedString.get());
-
     return addedString.release();
 }
 
index 0277ea222d101f7b0060abea6ee5bdde59c8f3b2..0091f38261544d45d82f5ce75c831c963ee314cf 100644 (file)
@@ -1,3 +1,14 @@
+2013-06-13  Ryosuke Niwa  <rniwa@webkit.org>
+
+        Remove LiteralIdentifierTable
+        https://bugs.webkit.org/show_bug.cgi?id=117613
+
+        Reviewed by Geoffrey Garen.
+
+        Removed LiteralIdentifierTable since it doesn't seem to have any perf. impact now.
+
+        * wtf/WTFThreadData.h:
+
 2013-06-13  Brent Fulgham  <bfulgham@apple.com>
 
         [WinCairo] Correct FeatureDefines.h Default for ENABLE_VIEW_MODE_CSS_MEDIA
index 1ab36ae168458f43fdf9f902c0f71a0f154bdfae..fa157e978967532762ad27ca1df1b09fecf5b564 100644 (file)
@@ -39,8 +39,6 @@
 // FIXME: This is a temporary layering violation while we move more string code to WTF.
 namespace JSC {
 
-typedef HashMap<const char*, RefPtr<StringImpl>, PtrHash<const char*> > LiteralIdentifierTable;
-
 class IdentifierTable {
     WTF_MAKE_FAST_ALLOCATED;
 public:
@@ -59,11 +57,8 @@ public:
         return true;
     }
 
-    LiteralIdentifierTable& literalTable() { return m_literalTable; }
-
 private:
     HashSet<StringImpl*> m_table;
-    LiteralIdentifierTable m_literalTable;
 };
 
 }