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

[David Sherwood via Phabricator via llvm-commits]

david-arm marked 2 inline comments as done.
david-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:3402
+                         .value_or(0);
+        if (Stride == -1)
+          return true;
----------------
sdesmalen wrote:
> Does it also need to return `true` for Strides < -1?
In practice if abs(Stride) > 1 we will always version the loop and only enter the vectorised loop if abs(Stride)==1, but checking for all values < 0 doesn't do any harm.


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

https://reviews.llvm.org/D146128