[PATCH] D90445: [SLP] Make SLPVectorizer to use `llvm.masked.gather` intrinsic

Anton Afanasyev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 17 00:13:11 PST 2020


anton-afanasyev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:1766
+                          ArrayRef<unsigned> ReorderIndices = None) {
+    assert(!(Bundle && EntryState == NeedToGather) &&
+           "Need to gather vectorized entry?");
----------------
ABataev wrote:
> What if `!Bundle && EntryState != NeedToGather`, i.e. trying to vectorize gather entry?
Thanks, added second assert.


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