Bug 141721

Summary:

StackLayoutPhase should use CodeBlock::usesArguments rather than FunctionExecutable::usesArguments

Product:

WebKit

Reporter:

Filip Pizlo <fpizlo>

Component:

JavaScriptCore

Assignee:

Filip Pizlo <fpizlo>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

rhodovan.u-szeged

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
the patch	msaboff: review+

Description Filip Pizlo 2015-02-17 13:05:23 PST

I've seen cases where the two are out of sync.  We know we can trust the CodeBlock::usesArguments because we use it everywhere else.

Comment 1 Filip Pizlo 2015-02-17 13:07:11 PST

Created attachment 246761 [details]
the patch

Comment 2 Michael Saboff 2015-02-17 13:33:09 PST

Comment on attachment 246761 [details]
the patch

r=me

Comment 3 Filip Pizlo 2015-02-17 13:41:55 PST

Landed in http://trac.webkit.org/changeset/180237

Comment 4 Brent Fulgham 2016-08-04 16:26:48 PDT

*** Bug 141246 has been marked as a duplicate of this bug. ***