[PATCH] D121199: [RISCV] Don't enable loop vectorizer interleaving if the V extension is scalable

[Liao Chunyu via Phabricator via llvm-commits]

liaolucy added a comment.





================
Comment at: llvm/lib/Target/RISCV/RISCVSubtarget.h:226
   unsigned getMaxInterleaveFactor() const {
-    return hasVInstructions() ? MaxInterleaveFactor : 1;
+    return useRVVForFixedLengthVectors() ? MaxInterleaveFactor : 1;
   }
----------------
kito-cheng wrote:
> `useRVVForFixedLengthVectors()` means we CAN use fixed vector but isn't means we CAN ONLY use fixed vector for vectorizer, so I think that would be vectorizer issue rather than backend's issue.
Thanks kito, any good modification suggestions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121199