Bug 81183 - NumericStrings should be inlined
Summary: NumericStrings should be inlined
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Benjamin Poulain
URL:
Keywords: EasyFix
Depends on:
Blocks:
 
Reported: 2012-03-14 19:04 PDT by Benjamin Poulain
Modified: 2012-03-15 15:01 PDT (History)
1 user (show)

See Also:

Attachments
Patch (2.51 KB, patch)
2012-03-14 19:09 PDT, Benjamin Poulain 		barraclough: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Poulain 2012-03-14 19:04:52 PDT 
For some reason, NumericStrings is not always inlined. When it is not, we loose a couple of percent when converting numbers to strings.
Comment 1 Benjamin Poulain 2012-03-14 19:09:04 PDT 
Created attachment 131975 [details]
Patch
Comment 2 Gavin Barraclough 2012-03-14 23:16:51 PDT 
Comment on attachment 131975 [details]
Patch

Nice catch!
Comment 3 Benjamin Poulain 2012-03-15 15:01:51 PDT 
Committed r110891: <http://trac.webkit.org/changeset/110891>