Bug 87179

Summary: DFG CSE should eliminate redundant WeakJSConstants
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch barraclough: review+

Filip Pizlo
Reported 2012-05-22 16:17:04 PDT
This will allow it to eliminate more redundant CheckStructures. This is only going to be a tiny win, since the bytecode parser usually eliminates these (it only misses those cases that are later revealed by CFG simplification). But it's an easy change, so it's worth it.
Attachments
the patch (2.43 KB, patch)
2012-05-22 16:25 PDT, Filip Pizlo 		barraclough: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2012-05-22 16:25:26 PDT
Created attachment 143393 [details] the patch
Filip Pizlo
Comment 2 2012-05-23 00:39:30 PDT
Landed in http://trac.webkit.org/changeset/118141
Filip Pizlo
Comment 3 2012-05-25 16:01:18 PDT
Merged in http://trac.webkit.org/changeset/118582
Note You need to log in before you can comment on or make changes to this bug.