[PATCH] D146128: [SVE][LoopVectorize] Add option to disable tail-folding for reverse loops

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 06:05:19 PDT 2023


david-arm marked an inline comment as done.
david-arm added a comment.

In D146128#4198864 <https://reviews.llvm.org/D146128#4198864>, @dmgreen wrote:

> LoopVectorizationLegality::containsDecreasingPointers seems to loop over all the instructions and call isConsecutivePtr, which just calls getPtrStride. Could that logic just be placed in AArch64TTIImpl::preferPredicateOverEpilogue? That is how it has worked in ARMTTIImpl::preferPredicateOverEpilogue via canTailPredicateLoop. Otherwise the code in containsDecreasingPointers is ran for any architecture, but only used by AArch64.

Done!


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

https://reviews.llvm.org/D146128



More information about the llvm-commits mailing list