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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 28 11:10:50 PST 2018

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;



More information about the llvm-commits mailing list