Bug 131899 - Arguments objects shouldn't need a destructor
Summary: Arguments objects shouldn't need a destructor
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Mark Hahnenberg
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-19 14:01 PDT by Mark Hahnenberg
Modified: 2014-04-21 18:38 PDT (History)
1 user (show)

See Also:

Attachments
Patch (20.24 KB, patch)
2014-04-21 14:33 PDT, Mark Hahnenberg 		no flags 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 Mark Hahnenberg 2014-04-19 14:01:29 PDT 
We can use CopiedSpace for their backing stores instead of malloc.
Comment 1 Mark Hahnenberg 2014-04-21 14:33:05 PDT 
Created attachment 229832 [details]
Patch
Comment 2 Mark Hahnenberg 2014-04-21 16:13:15 PDT 
This neutral on the JS benchmarks we track.
Comment 3 Mark Hahnenberg 2014-04-21 17:00:37 PDT 
This is a 1.5% speedup on dromaeo.
Comment 4 Mark Hahnenberg 2014-04-21 17:01:45 PDT 
(In reply to comment #3)
> This is a 1.5% speedup on dromaeo.

(See http://dromaeo.com/?id=220240,220241)
Comment 5 WebKit Commit Bot 2014-04-21 18:38:02 PDT 
Comment on attachment 229832 [details]
Patch

Clearing flags on attachment: 229832

Committed r167641: <http://trac.webkit.org/changeset/167641>
Comment 6 WebKit Commit Bot 2014-04-21 18:38:04 PDT 
All reviewed patches have been landed.  Closing bug.