[PATCH] D52327: [Loop Vectorizer] Abandon vectorization when no integer IV found

[Hideki Saito via Phabricator via llvm-commits]

hsaito added a comment.

Sorry, I totally forgot about my old patch  https://reviews.llvm.org/D47216.

1. I like your two different message version better than changing Line 788 condition to if (!WidestIndTy). That's good.
2. Please add non-NULL assertion after "Type *IdxTy = Legal->getWidestInductionType();" in InnerLoopVectorizer::getOrCreateTripCount().
3. Please include LIT test from https://reviews.llvm.org/D47216.

Thanks,
Hideki


https://reviews.llvm.org/D52327