[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