Page MenuHomePhabricator

[Unroll] When completely unrolling the loop, replace conditinal branches with unconditional.
ClosedPublic

Authored by mzolotukhin on Sep 17 2015, 4:05 PM.

Details

Summary

I see no reasons why we don't do this at the moment (or I'm misreading
the code). Later clean-ups replace conditional branches anyway, but
that's unnecessary work. It might be even more relevant if we unroll
loops with relatively high trip-count.

Does this patch make sense?

Event Timeline

mzolotukhin retitled this revision from to [Unroll] When completely unrolling the loop, replace conditinal branches with unconditional..
mzolotukhin updated this object.
mzolotukhin added a reviewer: hfinkel.
mzolotukhin added a subscriber: llvm-commits.
hfinkel edited edge metadata.Sep 23 2015, 1:20 AM

Yes, this seems to make sense.

This revision was automatically updated to reflect the committed changes.