diff options
| author | Chandler Carruth <chandlerc@gmail.com> | 2012-12-17 18:48:07 +0000 |
|---|---|---|
| committer | Chandler Carruth <chandlerc@gmail.com> | 2012-12-17 18:48:07 +0000 |
| commit | e3f4119b064da4bad36bb3f1b60c77e0ccef3081 (patch) | |
| tree | aab79250cbb5500b21622a50782f9c9bb8ef5564 /llvm/lib/Transforms/Scalar/SROA.cpp | |
| parent | beea120d6e10fa22fff7fe94bc961700d631dcba (diff) | |
| download | bcm5719-llvm-e3f4119b064da4bad36bb3f1b60c77e0ccef3081.tar.gz bcm5719-llvm-e3f4119b064da4bad36bb3f1b60c77e0ccef3081.zip | |
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.
llvm-svn: 170353
Diffstat (limited to 'llvm/lib/Transforms/Scalar/SROA.cpp')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/SROA.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Transforms/Scalar/SROA.cpp b/llvm/lib/Transforms/Scalar/SROA.cpp index ab61a2fd674..a0ee849a03f 100644 --- a/llvm/lib/Transforms/Scalar/SROA.cpp +++ b/llvm/lib/Transforms/Scalar/SROA.cpp @@ -2150,7 +2150,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 |
