diff options
author | Warren Ristow <warren.ristow@sony.com> | 2018-09-21 23:03:50 +0000 |
---|---|---|
committer | Warren Ristow <warren.ristow@sony.com> | 2018-09-21 23:03:50 +0000 |
commit | 4f27730eaf600aeeb855964de09b90ff168ae247 (patch) | |
tree | edecdba46da7a274b009db63fb98439d8b3fee47 /llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | |
parent | d7e83e73627c84f19b1dc3c2b5859035367356ae (diff) | |
download | bcm5719-llvm-4f27730eaf600aeeb855964de09b90ff168ae247.tar.gz bcm5719-llvm-4f27730eaf600aeeb855964de09b90ff168ae247.zip |
[Loop Vectorizer] Abandon vectorization when no integer IV found
Support for vectorizing loops with secondary floating-point induction
variables was added in r276554. A primary integer IV is still required
for vectorization to be done. If an FP IV was found, but no integer IV
was found at all (primary or secondary), the attempt to vectorize still
went forward, causing a compiler-crash. This change abandons that
attempt when no integer IV is found. (Vectorizing FP-only cases like
this, rather than bailing out, is discussed as possible future work
in D52327.)
See PR38800 for more information.
Differential Revision: https://reviews.llvm.org/D52327
llvm-svn: 342786
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/LoopVectorize.cpp')
-rw-r--r-- | llvm/lib/Transforms/Vectorize/LoopVectorize.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp index d5708d0c8f0..e1795c5e056 100644 --- a/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp +++ b/llvm/lib/Transforms/Vectorize/LoopVectorize.cpp @@ -2303,6 +2303,7 @@ Value *InnerLoopVectorizer::getOrCreateTripCount(Loop *L) { "Invalid loop count"); Type *IdxTy = Legal->getWidestInductionType(); + assert(IdxTy && "No type for induction"); // The exit count might have the type of i64 while the phi is i32. This can // happen if we have an induction variable that is sign extended before the |