RESOLVED INVALID197398
[B3] Tails of size 1 can be safely duplicated regardless of their number of predecessors 
https://bugs.webkit.org/show_bug.cgi?id=197398
Summary [B3] Tails of size 1 can be safely duplicated regardless of their number of p...
Robin Morisset
Reported 2019-04-29 17:59:21 PDT
Replacing a jump (one instruction) by a branch/switch/return (whatever the jump points to, also one instruction) cannot increase the number of instructions, and can be helpful to to things like foldPathConstants that run just after duplicateTails. In most cases we already do this, as the block will have few predecessors, but we should do it unconditionally. The benefit is almost certainly negligible (a quick experiment shows we only neglect to do this optimization 23 times on all of JetStream2), but the change is microscopic too (literally just one line).
Attachments
Patch (1.27 KB, patch)
2019-04-29 18:03 PDT, Robin Morisset 		rmorisset: review-
rmorisset: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Robin Morisset
Comment 1 2019-04-29 18:03:13 PDT
Created attachment 368524 [details] Patch
Robin Morisset
Comment 2 2019-04-29 18:14:04 PDT
Comment on attachment 368524 [details] Patch Forget it, I got confused between successors and predecessors, this check does not do at all what I thought.
Note You need to log in before you can comment on or make changes to this bug.