[PATCH] D25963: [LoopUnroll] Implement profile-based loop peeling

Dehao Chen via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 13:28:42 PST 2016


danielcdh added a comment.

In https://reviews.llvm.org/D25963#600188, @mkuper wrote:

> Rebased to resolve conflicts with r287186.
>
> Dehao, note I moved the flat loop check to be below the peeling computation, otherwise it would preempt it.
>  I think as long as it's above runtime unrolling, it does what you meant, right?


SGTM. Looks like the check becomes noop when peeling is enabled. But it would still serve as a backup plan when peeling is not enabled.


https://reviews.llvm.org/D25963





More information about the llvm-commits mailing list