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

[Igor Kudrin via Phabricator via llvm-commits]

ikudrin added a comment.

Our test was for AArch64, so there should be no effects like loop alignment, if I understand you right. I couldn't find a benchmark to test the change against, so I prepared an artificial one based on `LLVM test suite/SingleSource/Benchmarks/Adobe-C++/loop-unroll.cpp`. I saw the performance gain 10%-30% depending on the iterations count (on x86_64). But I understand that that test is probably biased, so if you recommend some commonly recognized public benchmark that might be suitable for the case, I would try it. As for a more deep analysis, could you share some examples so I can grab some ideas from them?


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

https://reviews.llvm.org/D123865