[PATCH] D104750: [LV] Support Interleaved Store Group With Gaps

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 25 13:01:05 PDT 2021


Ayal accepted this revision.
Ayal added a comment.
This revision is now accepted and ready to land.

This looks good to me, thanks!
Adding a minor final optional nit.



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5418
   // Check if masking is required.
   // A Group may need masking for one of two reasons: it resides in a block that
   // needs predication, or it was decided to use masking to deal with gaps.
----------------
nit: may want to update this comment to explain of three reasons, as they appear below, further elaborating the second reason.


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

https://reviews.llvm.org/D104750



More information about the llvm-commits mailing list