[PATCH] D145163: Add support for vectorization of interleaved memory accesses for scalable VF

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 09:58:33 PST 2023


luke added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:2751
+          StridedVec = Builder.CreateShuffleVector(NewLoads[Part], StrideMask,
+                                                   "strided.vec");
 
----------------
It's somehow possible to reach here with a scalable vector type if `TII->hasInterleavedLoad` returns false. 
Can we check somewhere inside the vectorizer cost model that if `hasInterleavedLoad` is false then we rule out any recipe with an interleave group for a scalable VF?


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

https://reviews.llvm.org/D145163



More information about the llvm-commits mailing list