[PATCH] D30710: [LV] Vectorize GEPs

[Elena Demikhovsky via Phabricator via llvm-commits]

delena accepted this revision.
delena added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:4749
+      if (VF > 1 && OrigLoop->hasLoopInvariantOperands(GEP)) {
+        // If we are vectorizing, but the GEP has only loop-invariant operands,
+        // the GEP we build (by only using vector-typed operands for
----------------
This "IF" should be dropped, if the GEP with all LoopInvariantOperands will be in the Scalars and the getVectorValue() will know to braodcast it.
Please add TODO to the comments.


https://reviews.llvm.org/D30710