Bug 134739

Summary: [ftlopt] Move Flush(SetLocal) store elimination to StrengthReductionPhase
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, ggaren, mark.lam, mhahnenberg, msaboff, oliver, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 134677    
Attachments:
Description Flags
the patch
none
the patch
none
the patch mhahnenberg: review+

Description Filip Pizlo 2014-07-08 12:41:56 PDT
I'm going to streamline CSE around clobberize(), and so Flush(SetLocal) store elimination wouldn't belong anymore.  It doesn't quite belong anywhere, which means that it belongs in StrengthReductionPhase.
Comment 1 Filip Pizlo 2014-07-08 12:50:50 PDT
Created attachment 234584 [details]
the patch

Not yet ready for review because I'm still testing it
Comment 2 Filip Pizlo 2014-07-08 13:25:59 PDT
Created attachment 234588 [details]
the patch
Comment 3 Filip Pizlo 2014-07-08 20:35:31 PDT
Created attachment 234614 [details]
the patch
Comment 4 Mark Hahnenberg 2014-07-09 10:50:18 PDT
Comment on attachment 234614 [details]
the patch

r=me
Comment 5 Filip Pizlo 2014-07-09 14:17:12 PDT
Landed in http://trac.webkit.org/changeset/170929