[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