[PATCH] D78502: [ARM][HWLoops] Create hardware loops for sibling loops

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello. Sorry for the delay.

It turns out this patch, when run over all our benchmarks, was actually making things worse not better! The setup cost of the loop counter (from the expanded SCEV) can end up taking longer than you save for loops with a low iteration count. And I don't have a great way to mitigate that at the moment.

It's still something we obviously need though, and it's not the fault of this patch exactly that it's getting worse. I'll get the patch committed and see if there's anything we can do about the problems at a later point.


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

https://reviews.llvm.org/D78502