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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:1166
+  if (Mask) {
+    Mask->reserve(VL.size());
+    for (unsigned i = 0; i < VL.size(); i++)
----------------
ABataev wrote:
> Actually `Mask` is a full copy of `UseOrder`, you don't need all that complex stuff here
Oops, no, `Mask` is not a copy of `UseOrder`
But you can create it much simpler:
```
for (unsigned I = 0, E = VL.size(); I < E; ++I)
  Mask[UseOrder[I]] = I;
```


Repository:
  rL LLVM

https://reviews.llvm.org/D36130