diff options
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/Analysis/ConstantFolding.cpp | 5 | ||||
-rw-r--r-- | llvm/lib/Analysis/TargetTransformInfo.cpp | 4 | ||||
-rw-r--r-- | llvm/lib/IR/Constants.cpp | 2 |
3 files changed, 6 insertions, 5 deletions
diff --git a/llvm/lib/Analysis/ConstantFolding.cpp b/llvm/lib/Analysis/ConstantFolding.cpp index 40302fb..2476dc5 100644 --- a/llvm/lib/Analysis/ConstantFolding.cpp +++ b/llvm/lib/Analysis/ConstantFolding.cpp @@ -3794,11 +3794,6 @@ static Constant *ConstantFoldScalableVectorCall( SplatOps.push_back(Op); continue; } - // TODO: Should getSplatValue return a poison scalar for a poison vector? - if (isa<PoisonValue>(Op)) { - SplatOps.push_back(PoisonValue::get(Op->getType()->getScalarType())); - continue; - } Constant *Splat = Op->getSplatValue(); if (!Splat) return nullptr; diff --git a/llvm/lib/Analysis/TargetTransformInfo.cpp b/llvm/lib/Analysis/TargetTransformInfo.cpp index 0f85739..2d053e5 100644 --- a/llvm/lib/Analysis/TargetTransformInfo.cpp +++ b/llvm/lib/Analysis/TargetTransformInfo.cpp @@ -886,6 +886,10 @@ TargetTransformInfo::getOperandInfo(const Value *V) { OperandValueKind OpInfo = OK_AnyValue; OperandValueProperties OpProps = OP_None; + // undef/poison don't materialize constants. + if (isa<UndefValue>(V)) + return {OK_AnyValue, OP_None}; + if (isa<ConstantInt>(V) || isa<ConstantFP>(V)) { if (const auto *CI = dyn_cast<ConstantInt>(V)) { if (CI->getValue().isPowerOf2()) diff --git a/llvm/lib/IR/Constants.cpp b/llvm/lib/IR/Constants.cpp index fa45330..a3c725b 100644 --- a/llvm/lib/IR/Constants.cpp +++ b/llvm/lib/IR/Constants.cpp @@ -1711,6 +1711,8 @@ void ConstantVector::destroyConstantImpl() { Constant *Constant::getSplatValue(bool AllowPoison) const { assert(this->getType()->isVectorTy() && "Only valid for vectors!"); + if (isa<PoisonValue>(this)) + return PoisonValue::get(cast<VectorType>(getType())->getElementType()); if (isa<ConstantAggregateZero>(this)) return getNullValue(cast<VectorType>(getType())->getElementType()); if (auto *CI = dyn_cast<ConstantInt>(this)) |