diff options
| author | Sven van Haastregt <sven.vanhaastregt@arm.com> | 2018-03-07 10:29:28 +0000 |
|---|---|---|
| committer | Sven van Haastregt <sven.vanhaastregt@arm.com> | 2018-03-07 10:29:28 +0000 |
| commit | 19f531d31ec8e21488b1852d9a99e0efef5f79f9 (patch) | |
| tree | 875772c0f3fe46e767d9a3b7b70f4b120771a07d /llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp | |
| parent | 9b996f03cbfca337e907369b05df1948732b9dbd (diff) | |
| download | bcm5719-llvm-19f531d31ec8e21488b1852d9a99e0efef5f79f9.tar.gz bcm5719-llvm-19f531d31ec8e21488b1852d9a99e0efef5f79f9.zip | |
[LoadStoreVectorizer] Differentiate between <1 x T> and T
The LoadStoreVectorizer thought that <1 x T> and T were the same types
when merging stores, leading to a crash later.
Patch by Erik Hogeman.
Differential Revision: https://reviews.llvm.org/D44014
llvm-svn: 326884
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp b/llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp index 1b8a79ad41b..f438aae9d1d 100644 --- a/llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp @@ -316,6 +316,7 @@ bool Vectorizer::isConsecutiveAccess(Value *A, Value *B) { Type *PtrATy = PtrA->getType()->getPointerElementType(); Type *PtrBTy = PtrB->getType()->getPointerElementType(); if (PtrA == PtrB || + PtrATy->isVectorTy() != PtrBTy->isVectorTy() || DL.getTypeStoreSize(PtrATy) != DL.getTypeStoreSize(PtrBTy) || DL.getTypeStoreSize(PtrATy->getScalarType()) != DL.getTypeStoreSize(PtrBTy->getScalarType())) |
