[Webkit-unassigned] [Bug 153981] [ES6] Arrow function syntax. Emit loading&putting this/super only if they are used in arrow function

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=153981

--- Comment #23 from GSkachkov <gskachkov at gmail.com> ---
(In reply to comment #20)
> (In reply to comment #17)
> > (In reply to comment #3)
> > > Comment on attachment 271327 [details]
> > > Patch
> > > 
> > > View in context:
> > > https://bugs.webkit.org/attachment.cgi?id=271327&action=review
> > > 
> > > LGTM, just a few design suggestions and renaming suggestions.
> > > Have you run perf numbers?
> > > 
> > 
> > I think it quite good figures for arrow function:
> >                                                       before_patch          
> > after_patch
> >    arrowfunction-call-in-class-constructor          300.1710+-13.5061    ^  
> > 191.4837+-4.5846        ^ definitely 1.5676x faster
> >    arrowfunction-call-in-class-method                91.2163+-7.7129     ^  
> > 29.3135+-10.2887       ^ definitely 3.1118x faster
> >    arrowfunction-call-in-function                    96.5411+-31.9533    ^  
> > 19.6813+-1.5141        ^ definitely 4.9052x faster
> >    arrowfunction-call                                33.9300+-3.3111     ^  
> > 27.0056+-0.9594        ^ definitely 1.2564x faster
> 
> Nice.
> 
> It would be worthwhile to open a bug to rewrite the deltablue benchmark
> (or some other benchmark that tests function calling) to use
> arrow functions everywhere. That way we can compare that performance
> to the performance of just using ordinary functions. And we can track
> performance in arrow functions w/ that benchmark.

I"ve added your comment to the issue that related to arrow function benchmark tests
https://bugs.webkit.org/show_bug.cgi?id=148034

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160222/9c7bb469/attachment.html>