Bug 156865 - Remove some unnecessary RefPtrs in the parser
Summary: Remove some unnecessary RefPtrs in the parser
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Saam Barati
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-21 13:54 PDT by Saam Barati
Modified: 2016-04-21 16:30 PDT (History)
11 users (show)

See Also:

Attachments
patch (10.24 KB, patch)
2016-04-21 14:04 PDT, Saam Barati 		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 Saam Barati 2016-04-21 13:54:47 PDT 
...
Comment 1 Oliver Hunt 2016-04-21 13:56:39 PDT 
Is this for binary size or performance?
Comment 2 Filip Pizlo 2016-04-21 13:58:00 PDT 
(In reply to comment #1)
> Is this for binary size or performance?

We're steadily pushing against parse time regressions from new ES6 features.
Comment 3 Saam Barati 2016-04-21 14:04:03 PDT 
Created attachment 276955 [details]
patch
Comment 4 Saam Barati 2016-04-21 14:05:23 PDT 
(In reply to comment #1)
> Is this for binary size or performance?

performance.
Comment 5 WebKit Commit Bot 2016-04-21 16:30:17 PDT 
Comment on attachment 276955 [details]
patch

Clearing flags on attachment: 276955

Committed r199845: <http://trac.webkit.org/changeset/199845>
Comment 6 WebKit Commit Bot 2016-04-21 16:30:23 PDT 
All reviewed patches have been landed.  Closing bug.