[PATCH] D26905: [SLP] Vectorize loads of consecutive memory accesses, accessed in non-consecutive (jumbled) way.

[Matthew Simpson via Phabricator via llvm-commits]

mssimpso added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:2575-2576
+      if (E->NeedToShuffle) {
+        assert(VL.size() == E->Scalars.size() &&
+               "Equal number of scalars expected");
+        SmallVector<Constant *, 8> Mask;
----------------
mssimpso wrote:
> ashahid wrote:
> > mssimpso wrote:
> > > Hi Shahid,
> > > 
> > > I'm hitting the assertion here while testing this patch. Can you take a look?
> > Sure. If possible can you share the asserting test?
> Sure, I'll try and reduce something for you.
OK, you should be able to reproduce the assert with the bugpoint reduced test case at P7950. Thanks!

```
opt < D26905.ll -slp-vectorizer -S
```


https://reviews.llvm.org/D26905