diff options
author | Dan Gohman <gohman@apple.com> | 2009-07-01 21:38:46 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2009-07-01 21:38:46 +0000 |
commit | 72d5fbb927d7e97bf5e170f977914269f3c41760 (patch) | |
tree | cab29373dffe9f84f4a4378c4d7f3e0a308cd645 /lib/Transforms | |
parent | 8c217fd2a27263a3d76d3fc053d32ea5ef2a2d9a (diff) | |
download | external_llvm-72d5fbb927d7e97bf5e170f977914269f3c41760.zip external_llvm-72d5fbb927d7e97bf5e170f977914269f3c41760.tar.gz external_llvm-72d5fbb927d7e97bf5e170f977914269f3c41760.tar.bz2 |
Fix an instcombine abort on a scalar-to-vector bitcast. This fixes PR4487.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@74646 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
-rw-r--r-- | lib/Transforms/Scalar/InstructionCombining.cpp | 16 |
1 files changed, 15 insertions, 1 deletions
diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp index 5bd17e0..16e5ce0 100644 --- a/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/lib/Transforms/Scalar/InstructionCombining.cpp @@ -1085,8 +1085,22 @@ Value *InstCombiner::SimplifyDemandedUseBits(Value *V, APInt DemandedMask, break; } case Instruction::BitCast: - if (!I->getOperand(0)->getType()->isInteger()) + if (!I->getOperand(0)->getType()->isIntOrIntVector()) return false; // vector->int or fp->int? + + if (const VectorType *DstVTy = dyn_cast<VectorType>(I->getType())) { + if (const VectorType *SrcVTy = + dyn_cast<VectorType>(I->getOperand(0)->getType())) { + if (DstVTy->getNumElements() != SrcVTy->getNumElements()) + // Don't touch a bitcast between vectors of different element counts. + return false; + } else + // Don't touch a scalar-to-vector bitcast. + return false; + } else if (isa<VectorType>(I->getOperand(0)->getType())) + // Don't touch a vector-to-scalar bitcast. + return false; + if (SimplifyDemandedBits(I->getOperandUse(0), DemandedMask, RHSKnownZero, RHSKnownOne, Depth+1)) return I; |