[PATCH] D30587: [LV] Delete unneeded scalar GEP creation code

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 18:18:07 PST 2017


mkuper added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:2979
+#endif
+    Ptr = Gep ? getScalarValue(Gep, 0, 0) : getScalarValue(Ptr, 0, 0);
   } else {
----------------
Do we really need to pay attention to the GEP anymore?
I mean, if we already scalarize everything correctly, wouldn't just always using "getScalarValue(Ptr, 0, 0)" be enough?


https://reviews.llvm.org/D30587





More information about the llvm-commits mailing list