Changeset 43457 in webkit for trunk/JavaScriptCore/runtime/UString.cpp

Timestamp:

May 10, 2009, 4:32:31 AM (16 years ago)

Author:

[email protected]

Message:

2009-05-10 Maciej Stachowiak <​[email protected]>

Reviewed by Alexey Proskuryakov.

• speedup dtoa/strtod

Added a bunch of inlining, and replaced malloc with stack allocation.

0.5% SunSpider speedup (7% on string-tagcloud).

• runtime/NumberPrototype.cpp: (JSC::integerPartNoExp): (JSC::numberProtoFuncToExponential):
• runtime/UString.cpp: (JSC::concatenate): (JSC::UString::from):
• wtf/dtoa.cpp: (WTF::BigInt::BigInt): (WTF::BigInt::operator=): (WTF::Balloc): (WTF::Bfree): (WTF::multadd): (WTF::s2b): (WTF::i2b): (WTF::mult): (WTF::pow5mult): (WTF::lshift): (WTF::cmp): (WTF::diff): (WTF::b2d): (WTF::d2b): (WTF::ratio): (WTF::strtod): (WTF::quorem): (WTF::freedtoa): (WTF::dtoa):
• wtf/dtoa.h:

File:

• trunk/JavaScriptCore/runtime/UString.cpp (modified) (4 diffs)

trunk/JavaScriptCore/runtime/UString.cpp

@@ -799 +799 @@
     int sign;
 
-    char* result = WTF::dtoa(d, 0, &decimalPoint, &sign, NULL);
+    char result[80];
+    WTF::dtoa(result, d, 0, &decimalPoint, &sign, NULL);
     int length = static_cast<int>(strlen(result));
   
@@ -850 +851 @@
     }
     
-  WTF::freedtoa(result);
-
-  return concatenate(rep, buf);
+    return concatenate(rep, buf);
 }
 
@@ -940 +939 @@
     int decimalPoint;
     int sign;
-
-    char* result = WTF::dtoa(d, 0, &decimalPoint, &sign, NULL);
+    
+    char result[80];
+    WTF::dtoa(result, d, 0, &decimalPoint, &sign, NULL);
     int length = static_cast<int>(strlen(result));
   
@@ -992 +992 @@
     }
     
-  WTF::freedtoa(result);
-
-  return UString(buf);
+    return UString(buf);
 }