[PATCH] D36130: [SLP] Vectorize jumbled memory loads.

[Shahid via Phabricator via llvm-commits]

ashahid added inline comments.


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:1660
+                            "permutation of loaded lanes.\n");
+            newTreeEntry(Sorted, /*Vectorized*/ true, UserTreeIdx,
+                         ReuseShuffleIndicies, Mask);
----------------
ABataev wrote:
> No, use original `VL` here, do not use `Sorted`. In this case you won't need an additional argument in `sortLoadAccesses` and you don't need all that complex stuff with the lambda on line 3528
If I am not wrong, for LOADs, VL0 must be the 1st element of the buffer whose base address will be used for vector load. 
So using VL will break this assumption.


Repository:
  rL LLVM

https://reviews.llvm.org/D36130