Reuse output array instead of creating a new one for each call in Date.parse

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@1544 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

diff --git a/src/date-delay.js b/src/date-delay.js
index d9b2421cc8e6f3ecb55b7c2ccdb123fc36b1ec58..2421e5baecc94c2602e9d6fafd4e500da2de0d1b 100644 (file)
--- a/src/date-delay.js
+++ b/src/date-delay.js
@@ -228,7 +228,6 @@ function TimeInYear(year) {
 
 
 // Compute modified Julian day from year, month, date.
-// The missing days in 1582 are ignored for JavaScript compatibility.
 function ToJulianDay(year, month, date) {
   var jy = (month > 1) ? year : year - 1;
   var jm = (month > 1) ? month + 2 : month + 14;
@@ -568,10 +567,12 @@ function DatePrintString(time) {
 
 // -------------------------------------------------------------------
 
+// Reused output buffer.
+var parse_buffer = $Array(7);
 
 // ECMA 262 - 15.9.4.2
 function DateParse(string) {
-  var arr = %DateParseString(ToString(string));
+  var arr = %DateParseString(ToString(string), parse_buffer);
   if (IS_NULL(arr)) return $NaN;
 
   var day = MakeDay(arr[0], arr[1], arr[2]);

diff --git a/src/dateparser-inl.h b/src/dateparser-inl.h
index ba5c0012054fd6c2bb68ac9f5a0ebdeefd5cc541..734c151d2559df691053183d9ff38ed901e8f4bf 100644 (file)
--- a/src/dateparser-inl.h
+++ b/src/dateparser-inl.h
@@ -29,7 +29,7 @@ namespace v8 { namespace internal {
 
 template <typename Char>
 bool DateParser::Parse(Vector<Char> str, FixedArray* out) {
-  ASSERT(out->length() == OUTPUT_SIZE);
+  ASSERT(out->length() >= OUTPUT_SIZE);
   InputReader<Char> in(str);
   TimeZoneComposer tz;
   TimeComposer time;

diff --git a/src/dateparser.h b/src/dateparser.h
index b03dd8df4d2ff01bb2bbcac656d30bc2008238ab..04d7e8baf17d69b7817c70af2f5cabcc0fbda7ff 100644 (file)
--- a/src/dateparser.h
+++ b/src/dateparser.h
@@ -48,7 +48,9 @@ class DateParser : public AllStatic {
   template <typename Char>
   static bool Parse(Vector<Char> str, FixedArray* output);
 
-  enum {YEAR, MONTH, DAY, HOUR, MINUTE, SECOND, UTC_OFFSET, OUTPUT_SIZE};
+  enum {
+    YEAR, MONTH, DAY, HOUR, MINUTE, SECOND, UTC_OFFSET, OUTPUT_SIZE
+  };
 
  private:
   // Range testing

diff --git a/src/runtime.cc b/src/runtime.cc
index 89f524ce7e9811d516d50d99090c89f59700ac7b..06ef46acdb6658c7dc4312789895817c304377b6 100644 (file)
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -4480,27 +4480,30 @@ static Object* Runtime_DateCurrentTime(Arguments args) {
 
 static Object* Runtime_DateParseString(Arguments args) {
   HandleScope scope;
-  ASSERT(args.length() == 1);
-
-  CONVERT_CHECKED(String, string_object, args[0]);
+  ASSERT(args.length() == 2);
 
-  Handle<String> str(string_object);
+  CONVERT_ARG_CHECKED(String, str, 0);
   FlattenString(str);
-  Handle<FixedArray> output = Factory::NewFixedArray(DateParser::OUTPUT_SIZE);
+
+  CONVERT_ARG_CHECKED(JSArray, output, 1);
+  RUNTIME_ASSERT(output->HasFastElements());
+
+  AssertNoAllocation no_allocation;
+
+  FixedArray* output_array = output->elements();
+  RUNTIME_ASSERT(output_array->length() >= DateParser::OUTPUT_SIZE);
   bool result;
-  {
-    AssertNoAllocation no_allocation;
-    if (StringShape(*str).IsAsciiRepresentation()) {
-      result = DateParser::Parse(str->ToAsciiVector(), *output);
-    } else {
-      ASSERT(StringShape(*str).IsTwoByteRepresentation());
-      result = DateParser::Parse(str->ToUC16Vector(), *output);
-    }
+  if (StringShape(*str).IsAsciiRepresentation()) {
+    result = DateParser::Parse(str->ToAsciiVector(), output_array);
+  } else {
+    ASSERT(StringShape(*str).IsTwoByteRepresentation());
+    result = DateParser::Parse(str->ToUC16Vector(), output_array);
   }
+
   if (result) {
-    return *Factory::NewJSArrayWithElements(output);
+    return *output;
   } else {
-    return *Factory::null_value();
+    return Heap::null_value();
   }
 }
 

diff --git a/src/runtime.h b/src/runtime.h
index 4b5a267e77ec232441fc6852b08f03db81c240ec..0a4795d750209418fa18ec1db8296981d7e9b3e8 100644 (file)
--- a/src/runtime.h
+++ b/src/runtime.h
@@ -185,7 +185,7 @@ namespace v8 { namespace internal {
   \
   /* Dates */ \
   F(DateCurrentTime, 0) \
-  F(DateParseString, 1) \
+  F(DateParseString, 2) \
   F(DateLocalTimezone, 1) \
   F(DateLocalTimeOffset, 0) \
   F(DateDaylightSavingsOffset, 1) \