Title: [170857] trunk/Source/_javascript_Core
Revision
170857
Author
akl...@apple.com
Date
2014-07-07 14:06:15 -0700 (Mon, 07 Jul 2014)

Log Message

Fast path for jsStringWithCache() when asked for the same string repeatedly.
<https://webkit.org/b/134635>

Reviewed by Darin Adler.

Follow-up to r170818 addressing a review comment by Geoff Garen.

* runtime/JSString.cpp:
(JSC::jsStringWithCacheSlowCase):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (170856 => 170857)


--- trunk/Source/_javascript_Core/ChangeLog	2014-07-07 20:52:49 UTC (rev 170856)
+++ trunk/Source/_javascript_Core/ChangeLog	2014-07-07 21:06:15 UTC (rev 170857)
@@ -1,3 +1,15 @@
+2014-07-07  Andreas Kling  <akl...@apple.com>
+
+        Fast path for jsStringWithCache() when asked for the same string repeatedly.
+        <https://webkit.org/b/134635>
+
+        Reviewed by Darin Adler.
+
+        Follow-up to r170818 addressing a review comment by Geoff Garen.
+
+        * runtime/JSString.cpp:
+        (JSC::jsStringWithCacheSlowCase):
+
 2014-07-07  Tibor Meszaros  <tmeszaros.u-sze...@partner.samsung.com>
 
         Add missing ENABLE(FTL_JIT) guards

Modified: trunk/Source/_javascript_Core/runtime/JSString.cpp (170856 => 170857)


--- trunk/Source/_javascript_Core/runtime/JSString.cpp	2014-07-07 20:52:49 UTC (rev 170856)
+++ trunk/Source/_javascript_Core/runtime/JSString.cpp	2014-07-07 21:06:15 UTC (rev 170857)
@@ -380,7 +380,7 @@
     if (addResult.isNewEntry)
         addResult.iterator->value = jsString(&vm, String(stringImpl));
     vm.lastCachedString = addResult.iterator->value.get();
-    return vm.lastCachedString.get();
+    return addResult.iterator->value.get();
 }
 
 } // namespace JSC
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to