[PATCH] D30211: [LV] Merge floating point and integer induction widening code

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 04:29:26 PST 2017


delena added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:4741
     // Handle the pointer induction variable case.
     assert(P->getType()->isPointerTy() && "Unexpected type.");
     // This is the normalized GEP that starts counting at zero.
----------------
mssimpso wrote:
> delena wrote:
> > Is the pointer induction really different from int and FP induction?
> > May be you can put everything under widenInduction?
> It's not, and I'm planning to work on this soon. Thanks!
I'd put them all in one patch and call the function widenInduction(). but it is not a stumbling-block anyway.


https://reviews.llvm.org/D30211





More information about the llvm-commits mailing list