[PATCH] D91919: [SLP] Make SLPVectorizer to use `llvm.masked.scatter` intrinsic

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 04:57:37 PST 2020


anton-afanasyev planned changes to this revision.
anton-afanasyev added a comment.

In D91919#2439339 <https://reviews.llvm.org/D91919#2439339>, @RKSimon wrote:

>> Actually NFC now, since stores could be only seed entry, but we collect only consecutive stores there for now. It will be changed in future commits.
>
> Are you looking at supporting non-consecutive stores? This patch probably shouldn't be reviewed until we actually have something that uses and tests it.

Yes, I've planned to add such stores generation at the next step. But you're right, better to do this together with this step.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91919



More information about the llvm-commits mailing list