RESOLVED FIXED 125538
Simplify CSE's treatment of NodeRelevantToOSR 
https://bugs.webkit.org/show_bug.cgi?id=125538
Summary Simplify CSE's treatment of NodeRelevantToOSR
Filip Pizlo
Reported 2013-12-10 13:14:12 PST
CSE does a simplification over Phantoms based on whether the source of the Phantom is relevant to OSR. It does this in a way that totally makes sense if node references are block-local, and I can sort of convince myself that it kind of ought to sort of work for SSA, but it's confusing in its current form. We should simplify this so that it obviously works: if there is any MovHint on a node then the node is relevant to OSR.
Attachments
the patch (4.43 KB, patch)
2013-12-10 13:16 PST, Filip Pizlo 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2013-12-10 13:16:03 PST
Created attachment 218900 [details] the patch
Filip Pizlo
Comment 2 2013-12-10 18:07:56 PST
Landed in http://trac.webkit.org/changeset/160407
Note You need to log in before you can comment on or make changes to this bug.