[PATCH] D123865: [LoopPeel] Allow partial unrolling for profile-based peeling

[David Li via Phabricator via llvm-commits]

davidxl added a comment.

Have you analyzed the root cause of the performance regression (with performance counters)? Is it due to other missed optimizations without the peeling or is it due to side effects such as loop alignment?

Changes like this one is very likely to cause regressions in other workload, so it might better to solve the problem with tuning options (which are there for a reason). Longer term, I'd like to see more sophisticated cost/benefit analysis implemented which will also be target dependent.


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

https://reviews.llvm.org/D123865