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

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello - yeah I was thinking of 2, as isConsecutivePtr is fairly simple. The Arm backend uses getPtrStride and is more careful about the strides it allows.  LD2/LD4 are ruled out, but other strides are allowed to use the gather/scatters for example. It will miss the SymbolicStrides and "Assume" checks though.

I have no strong opinion - it was just a suggestion about how it has worked in other architectures.  I'll leave it to you what you think is best.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146128