[PATCH] D30710: [LV] Vectorize GEPs

[Michael Kuperstein via Phabricator via llvm-commits]

mkuper added a comment.

Sorry for the delay, Matt.

This makes sense to me.
It was reasonable to special-case GEPs when vectorizing them was mostly pointless, because the vast majority of uses would be scalar. But I think the infrastructure now is good enough to handle it in a generic way. Thanks!

Is there a test that actually stores a vector of pointers?



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:5449
+
+      // If there's no pointer operand, there's nothing to do.
+      auto *Ptr = dyn_cast_or_null<Instruction>(getPointerOperand(&I));
----------------
nit: "If there's no pointer operand or the pointer operand is not an instruction"


https://reviews.llvm.org/D30710