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

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 19:49:10 PST 2017


mkuper added subscribers: wmi, dberlin.
mkuper added inline comments.


================
Comment at: lib/Analysis/LoopAccessAnalysis.cpp:1075
+    APInt Offset(PtrBitWidth, 0);
+    Ptr->stripAndAccumulateInBoundsConstantOffsets(DL, Offset);
+    OffValPair.push_back(std::make_pair(Offset.getSExtValue(), Val));
----------------
Thanks to @dberlin and @wmi - I now realize this is too simplistic.
It will handle cases where the offsets are constant, but not when all the offsets are variable, but the variables have constant differences from each other.

Anyway that can be handled separately. Could you add a FIXME here, please?


https://reviews.llvm.org/D26905





More information about the llvm-commits mailing list