diff options
| author | dim <dim@FreeBSD.org> | 2012-12-22 20:16:21 +0000 |
|---|---|---|
| committer | dim <dim@FreeBSD.org> | 2012-12-22 20:16:21 +0000 |
| commit | de145fce99c455d8d88617c0033a9e632285a4be (patch) | |
| tree | b12c63a4639709d47abe1752f841539870fdef80 /contrib/llvm/lib | |
| parent | 45679ebc7bd01bdb686bd33c1997dddb0f092ea5 (diff) | |
| download | FreeBSD-src-de145fce99c455d8d88617c0033a9e632285a4be.zip FreeBSD-src-de145fce99c455d8d88617c0033a9e632285a4be.tar.gz | |
Pull in r170353 from upstream llvm trunk:
Fix another SROA crasher, PR14601.
This was a silly oversight, we weren't pruning allocas which were used
by variable-length memory intrinsics from the set that could be widened
and promoted as integers. Fix that.
This should fix the following assertion failure:
Assertion failed: (CanSROA), function visitUsers, file
/usr/src/lib/clang/libllvmscalaropts/../../../contrib/llvm/lib/Transforms/Scalar/SROA.cpp,
line 2395.
Reported by: gerald
Diffstat (limited to 'contrib/llvm/lib')
| -rw-r--r-- | contrib/llvm/lib/Transforms/Scalar/SROA.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/llvm/lib/Transforms/Scalar/SROA.cpp b/contrib/llvm/lib/Transforms/Scalar/SROA.cpp index ccc2f7a..959b4e7 100644 --- a/contrib/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/contrib/llvm/lib/Transforms/Scalar/SROA.cpp @@ -2224,7 +2224,7 @@ static bool isIntegerWideningViable(const DataLayout &TD, !canConvertValue(TD, ValueTy, AllocaTy)) return false; } else if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(I->U->getUser())) { - if (MI->isVolatile()) + if (MI->isVolatile() || !isa<Constant>(MI->getLength())) return false; if (MemTransferInst *MTI = dyn_cast<MemTransferInst>(I->U->getUser())) { const AllocaPartitioning::MemTransferOffsets &MTO |
