[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