Bug 185126

Summary: LICM shouldn't hoist nodes if hoisted nodes exited in that code block
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch saam: review+

Filip Pizlo
Reported 2018-04-29 18:05:00 PDT
We allow speculative hoisting, but only once per codeBlock.
Attachments
the patch (3.10 KB, patch)
2018-04-29 18:14 PDT, Filip Pizlo 		saam: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2018-04-29 18:14:58 PDT
Created attachment 339102 [details] the patch
Saam Barati
Comment 2 2018-04-30 09:21:00 PDT
Comment on attachment 339102 [details] the patch r=me
Filip Pizlo
Comment 3 2018-04-30 17:05:02 PDT
Landed in https://trac.webkit.org/changeset/231185/webkit
Radar WebKit Bug Importer
Comment 4 2018-04-30 17:08:04 PDT
<rdar://problem/39855250>
Note You need to log in before you can comment on or make changes to this bug.