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

Fri Mar 24 11:40:29 PDT 2023

dmgreen added a comment.

In D146128#4209402 <https://reviews.llvm.org/D146128#4209402>, @david-arm wrote:

> 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!

Thanks

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

https://reviews.llvm.org/D146128