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

[Evgeniy via Phabricator via llvm-commits]

ebrevnov added a comment.

Hi @Carrot,

I checked your patch on original test case and it does resolve the issue. Unfortunately, we have 4 other test cases regressed from 1% to 5%.  Bad news is that we don't have sources for these cases. At glance they don't involve operations with floats.  So that is not a surprise they are not affected. I can continue investigation and try to provide short reproducer if needed. My fear here is that original change touches very wide class of cases and we won't be able to fix all of them one by one (or it will take too much time). Can we think of a more realistic approach to deal with regressions?

Thanks
Evgeniy


Repository:
  rL LLVM

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

https://reviews.llvm.org/D43256