[PATCH] D89048: [ARM][LowOverheadLoops] Insert loop start at end of block in more cases

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 20 00:56:23 PDT 2020


dmgreen added a comment.

> And I can't see the latency of DLS having any real affect on the performance of a loop, ..

Unfortunately I'm not sure that is always true exactly, but I'm not against the general idea of moving the loop start closer loop, especially for DLSTP. If it simplifies tail predication and makes it more reliable, then that's certainly a good thing.

I have reverted the patch that this depends on (38f625d0d1360b0 <https://reviews.llvm.org/rG38f625d0d1360b035271422bab922d22ed04d79a>) because it's had issues for a while now and we could do with it being correct so that we have a better place work work from. That way we can re-do this whilst having a higher confidence that we are not introducing bugs.


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

https://reviews.llvm.org/D89048



More information about the llvm-commits mailing list