RESOLVED FIXED 15350
skip extra hash lookup and avoid converting char* to UString for 19% speedup on CK JS array test 
https://bugs.webkit.org/show_bug.cgi?id=15350
Summary skip extra hash lookup and avoid converting char* to UString for 19% speedup ...
Maciej Stachowiak
Reported 2007-10-02 16:08:22 PDT
The Celtic Kane JavaScript Benchmark "Array" test can be sped up 19% by using the return value from add() to avoid an extra hash lookup for contains(), and by storing some strings in function-scope static UString variables instead of as constant strings used to make new UStrings every time. These issues mainly affect the Array "join" and "toString" operations, which unfortunately seem to make up the vast majority of time on this benchmark.
Attachments
patch implementing these two optimizations (1.48 KB, patch)
2007-10-02 16:30 PDT, Maciej Stachowiak 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Maciej Stachowiak
Comment 1 2007-10-02 16:30:12 PDT
Created attachment 16509 [details] patch implementing these two optimizations
Oliver Hunt
Comment 2 2007-10-02 16:37:17 PDT
Comment on attachment 16509 [details] patch implementing these two optimizations ooh, nice and simple
Note You need to log in before you can comment on or make changes to this bug.