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

[Hans Wennborg via Phabricator via llvm-commits]

hans added a comment.

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?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D43256