Bug 131636

Summary: Function.bind itself is too slow
Product: WebKit Reporter: Oliver Hunt <oliver>
Component: New BugsAssignee: Oliver Hunt <oliver>
Status: REOPENED ---    
Severity: Normal CC: commit-queue, ggaren
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 131666    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch ggaren: review+

Description Oliver Hunt 2014-04-14 14:59:09 PDT
Function.bind itself is too slow
Comment 1 Oliver Hunt 2014-04-14 15:01:14 PDT
Created attachment 229307 [details]
Patch
Comment 2 Oliver Hunt 2014-04-14 15:01:46 PDT
Created attachment 229308 [details]
Patch
Comment 3 Geoffrey Garen 2014-04-14 15:03:24 PDT
Comment on attachment 229308 [details]
Patch

r=me
Comment 4 Oliver Hunt 2014-04-14 15:06:11 PDT
Committed r167272: <http://trac.webkit.org/changeset/167272>
Comment 5 Alexey Proskuryakov 2014-04-14 23:10:57 PDT
This change broke multiple tests, and EWS saw that. Please do click on yellow bubbles before landing, even if they didn't turn red yet.

Broken tests:

js/dom/function-bind.html
js/regress/function-bind-create.html
fast/canvas/webgl/typed-arrays-in-workers.html
webgl/1.0.2/conformance/typedarrays/typed-arrays-in-workers.html

Will roll out.
Comment 6 WebKit Commit Bot 2014-04-14 23:11:44 PDT
Re-opened since this is blocked by bug 131666
Comment 7 Geoffrey Garen 2014-04-15 00:14:11 PDT
perf.webkit.org measured a 1% DYEB regression from this patch.