Bug 83554

Summary: DFG should flush SetLocals to arguments
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: ggaren
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
the patch
oliver: review+
the patch
buildbot: commit-queue-
the patch barraclough: review+

Description Filip Pizlo 2012-04-10 01:18:53 PDT
This is necessary to match baseline JIT argument capture behavior.
Comment 1 Filip Pizlo 2012-04-10 01:23:51 PDT
Created attachment 136412 [details]
the patch
Comment 2 Geoffrey Garen 2012-04-10 08:30:10 PDT
Comment 3 Filip Pizlo 2012-04-10 15:27:28 PDT
Created attachment 136557 [details]
the patch
Comment 4 Build Bot 2012-04-10 15:36:19 PDT
Comment on attachment 136557 [details]
the patch

Attachment 136557 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/12380546
Comment 5 Filip Pizlo 2012-04-10 15:46:38 PDT
Created attachment 136559 [details]
the patch
Comment 6 Gavin Barraclough 2012-04-10 16:15:35 PDT
Comment on attachment 136559 [details]
the patch

ArgumentPosition isn't a position, so I'd suggest a different name, but since I don't have a better one, R+ either way.
Comment 7 Filip Pizlo 2012-04-10 17:37:43 PDT
Landed in http://trac.webkit.org/changeset/113796