Bug 156389

Summary: Add IC support for arguments.length
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, keith_miller, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch
fpizlo: review-
the patch ggaren: review+

Description Filip Pizlo 2016-04-07 21:00:18 PDT
Patch forthcoming.
Comment 1 Filip Pizlo 2016-04-08 11:02:47 PDT
Created attachment 276016 [details]
the patch
Comment 2 Filip Pizlo 2016-04-08 11:03:11 PDT
Comment on attachment 276016 [details]
the patch

Ugh, never mind.
Comment 3 Filip Pizlo 2016-04-08 11:07:08 PDT
Created attachment 276018 [details]
the patch
Comment 4 Geoffrey Garen 2016-04-08 11:18:01 PDT
Comment on attachment 276018 [details]
the patch

r=me
Comment 5 Filip Pizlo 2016-04-08 11:23:53 PDT
I'm still testing this, it'll be a while.  I'm worried that I messed up DFG's profiling of .length, so I'll test everything.
Comment 6 Filip Pizlo 2016-04-08 12:37:26 PDT
Landed in http://trac.webkit.org/changeset/199240