Bug 156389

Summary:

Add IC support for arguments.length

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, keith_miller, mark.lam, msaboff, saam

Priority:

Version:

WebKit Nightly Build

Hardware:

All

OS:

All

Attachments:

Description	Flags
the patch	fpizlo: review-
the patch	ggaren: review+

Filip Pizlo

Reported 2016-04-07 21:00:18 PDT

Patch forthcoming.

Attachments
the patch (67.39 KB, patch) 2016-04-08 11:02 PDT, Filip Pizlo	fpizlo: review-	Details Formatted Diff Diff
the patch (35.14 KB, patch) 2016-04-08 11:07 PDT, Filip Pizlo	ggaren: review+	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Filip Pizlo

Comment 1 2016-04-08 11:02:47 PDT

Filip Pizlo

Comment 2 2016-04-08 11:03:11 PDT

Comment on attachment 276016 [details] the patch Ugh, never mind.

Filip Pizlo

Comment 3 2016-04-08 11:07:08 PDT

Geoffrey Garen

Comment 4 2016-04-08 11:18:01 PDT

Comment on attachment 276018 [details] the patch r=me

Filip Pizlo

Comment 5 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.

Filip Pizlo

Comment 6 2016-04-08 12:37:26 PDT

Note You need to log in before you can comment on or make changes to this bug.