[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