[PATCH] D46191: [LV] Preserve inbounds on created GEPs

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 30 17:07:58 PDT 2018


hsaito added a comment.

LGTM. Thanks.



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:2427
+      PartPtr = cast<GetElementPtrInst>(
+          Builder.CreateGEP(Ptr, Builder.getInt32(-Part * VF)));
+      PartPtr->setIsInBounds(InBounds);
----------------
Changing this to single GEP will enable further clean up, but I think I already asked a lot of clean up and you delivered a lot. If this bothers me, I'll do it myself.


Repository:
  rL LLVM

https://reviews.llvm.org/D46191





More information about the llvm-commits mailing list