RESOLVED FIXED 118778
fourthTier: Add a phase to create loop pre-headers 
https://bugs.webkit.org/show_bug.cgi?id=118778
Summary fourthTier: Add a phase to create loop pre-headers
Filip Pizlo
Reported 2013-07-16 22:18:36 PDT
Patch forthcoming.
Attachments
the patch (20.29 KB, patch)
2013-07-16 22:24 PDT, Filip Pizlo 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2013-07-16 22:24:48 PDT
Created attachment 206854 [details] the patch
Geoffrey Garen
Comment 2 2013-07-17 13:23:52 PDT
Comment on attachment 206854 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=206854&action=review > Source/JavaScriptCore/dfg/DFGNaturalLoops.h:63 > + // a natural loop. It's faster to call The suspense is killing me. Faster to call what?
Filip Pizlo
Comment 3 2013-07-17 14:02:13 PDT
Comment on attachment 206854 [details] the patch View in context: https://bugs.webkit.org/attachment.cgi?id=206854&action=review >> Source/JavaScriptCore/dfg/DFGNaturalLoops.h:63 >> + // a natural loop. It's faster to call > > The suspense is killing me. Faster to call what? Lol. It's faster to call NaturalLoops::belongsTo(). I'll fix that comment.
Filip Pizlo
Comment 4 2013-07-17 22:15:09 PDT
Landed in http://trac.webkit.org/changeset/152829
Note You need to log in before you can comment on or make changes to this bug.