Changeset 50705 in webkit for trunk/JavaScriptCore/runtime/DateConversion.cpp

Timestamp:

Nov 9, 2009, 7:14:26 PM (16 years ago)

Author:

[email protected]

Message:

Added a tiny cache for Date parsing.

Reviewed by Sam "Home Wrecker" Weinig.

SunSpider says 1.2% faster.

• runtime/DateConversion.cpp:

(JSC::parseDate): Try to reuse the last parsed Date, if present.

• runtime/JSGlobalData.cpp:

(JSC::JSGlobalData::resetDateCache):

• runtime/JSGlobalData.h: Added storage for last parsed Date. Refactored

this code to make resetting the date cache easier.

• runtime/JSGlobalObject.h:

(JSC::DynamicGlobalObjectScope::DynamicGlobalObjectScope): Updated for
refactoring.

• wtf/DateMath.cpp:

(JSC::parseDateFromNullTerminatedCharacters):

• wtf/DateMath.h: Changed ExecState to be first parameter, as is the JSC custom.

File:

• trunk/JavaScriptCore/runtime/DateConversion.cpp (modified) (2 diffs)

trunk/JavaScriptCore/runtime/DateConversion.cpp

@@ -44 +44 @@
 #include "DateConversion.h"
 
+#include "CallFrame.h"
 #include "UString.h"
 #include <wtf/DateMath.h>
@@ -54 +55 @@
 double parseDate(ExecState* exec, const UString &date)
 {
-    return parseDateFromNullTerminatedCharacters(date.UTF8String().c_str(), exec);
+    if (date == exec->globalData().cachedDateString)
+        return exec->globalData().cachedDateStringValue;
+    double value = parseDateFromNullTerminatedCharacters(exec, date.UTF8String().c_str());
+    exec->globalData().cachedDateString = date;
+    exec->globalData().cachedDateStringValue = value;
+    return value;
 }