[PATCH] D43256: [MBP] Move a latch block with conditional exit and multi predecessors to top of loop

Fri Jul 26 15:50:32 PDT 2019

Carrot added a comment.

In D43256#1602841 <https://reviews.llvm.org/D43256#1602841>, @hans wrote:

> In D43256#1550842 <https://reviews.llvm.org/D43256#1550842>, @Carrot wrote:
>
> > In D43256#1545519 <https://reviews.llvm.org/D43256#1545519>, @samparker wrote:
> >
> > > Do you have any benchmark numbers to show that this is generally profitable? From our downstream testing, it is not clear that this change is beneficial.
> >
> >
> > We got performance improvement in our internal search benchmark.
>
>
> How does this transformation impact the benchmark when not using profile data?

Benchmark is running, will report the result once it is finished.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D43256/new/

https://reviews.llvm.org/D43256