[PATCH] D36258: Disable loop peeling during full unrolling pass.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 22:28:30 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D36258#878507, @reames wrote:

> FYI, this patch causes a somewhat serious regression on one of our internal benchmarks.  Given we have a custom pass pipeline, it's definitely our responsibility to adapt to upstream changes not the other way around, but it would have helped us recognize the problem if either of two things had happened along with this patch:
>
> 1. The patch had been separated into two pieces: adding the new control know without changing the default and then a small patch changing the default.
> 2. The change had been announced in some way.  An email to llvm-dev probably would have been sufficient.
>
>   Given we're hardly the only ones with custom pass pipelines, I'd encourage everyone involved (author, reviewers, etc..) to think about the upgrade path in similar cases which arise in the future.


That's clearly interesting. This change, however, seems like it should be more-or-less a nop. If you're going to fully unroll, why would peeling help? Do you end up with a shorter critical path through the branches if we "peel" first?


Repository:
  rL LLVM

https://reviews.llvm.org/D36258





More information about the llvm-commits mailing list