[PATCH] D90445: [SLP] Make SLPVectorizer to use `llvm.masked.gather` intrinsic
Anton Afanasyev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 9 01:40:34 PST 2020
anton-afanasyev marked an inline comment as done.
anton-afanasyev added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:1581
+ /// Boolean value indicating that pointer operands are scattered.
+ bool IsScatteredOps = false;
+
----------------
dtemirbulatov wrote:
> Just to avoid any confusion, I think it is to change the name to IsScatterGatherOps?
Current `IsScatteredOps` means that _operands_ of load/store are _scattered_, so need either to gather, or to scatter them. I'm to change name to `IsScatterOpds` to avoid confusion with `Operations`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D90445/new/
https://reviews.llvm.org/D90445
More information about the llvm-commits
mailing list