Bug 159170

Summary: Clean up resetting reachability in B3/Air
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, keith_miller, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch ggaren: review+

Filip Pizlo
Reported 2016-06-27 14:47:06 PDT
In r202512 I fixed a bug in B3::Procedure::resetReachability(), but this fix was more complicated than need be. There was one set of data structures for the set of live basic blocks, and another for the set of live values - even though the set of blocks determined the set of values.
Attachments
the patch (6.28 KB, patch)
2016-06-27 15:02 PDT, Filip Pizlo 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2016-06-27 15:02:29 PDT
Created attachment 282171 [details] the patch
Geoffrey Garen
Comment 2 2016-06-27 15:12:08 PDT
Comment on attachment 282171 [details] the patch r=me
Filip Pizlo
Comment 3 2016-06-27 15:27:13 PDT
Landed in https://trac.webkit.org/changeset/202518
Note You need to log in before you can comment on or make changes to this bug.