[llvm-commits] [llvm] r148906 - in /llvm/trunk: include/llvm/Constants.h lib/VMCore/Constants.cpp
Argyrios Kyrtzidis
akyrtzi at gmail.com
Tue Jan 24 18:42:41 PST 2012
Author: akirtzidis
Date: Tue Jan 24 20:42:41 2012
New Revision: 148906
URL: http://llvm.org/viewvc/llvm-project?rev=148906&view=rev
Log:
Revert r148901 because it crashes llvm tests.
Original log:
Introduce a new ConstantVector::getSplat constructor function to
simplify a really common case.
Modified:
llvm/trunk/include/llvm/Constants.h
llvm/trunk/lib/VMCore/Constants.cpp
Modified: llvm/trunk/include/llvm/Constants.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Constants.h?rev=148906&r1=148905&r2=148906&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Constants.h (original)
+++ llvm/trunk/include/llvm/Constants.h Tue Jan 24 20:42:41 2012
@@ -489,10 +489,6 @@
// ConstantVector accessors
static Constant *get(ArrayRef<Constant*> V);
- /// getSplat - Return a ConstantVector with the specified constant in each
- /// element.
- static Constant *getSplat(unsigned NumElts, Constant *Elt);
-
/// Transparently provide more efficient getOperand methods.
DECLARE_TRANSPARENT_OPERAND_ACCESSORS(Constant);
@@ -761,11 +757,6 @@
static Constant *get(LLVMContext &Context, ArrayRef<float> Elts);
static Constant *get(LLVMContext &Context, ArrayRef<double> Elts);
- /// getSplat - Return a ConstantVector with the specified constant in each
- /// element. The specified constant has to be a of a compatible type (i8/i16/
- /// i32/i64/float/double) and must be a ConstantFP or ConstantInt.
- static Constant *getSplat(unsigned NumElts, Constant *Elt);
-
/// getType - Specialize the getType() method to always return a VectorType,
/// which reduces the amount of casting needed in parts of the compiler.
///
Modified: llvm/trunk/lib/VMCore/Constants.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Constants.cpp?rev=148906&r1=148905&r2=148906&view=diff
==============================================================================
--- llvm/trunk/lib/VMCore/Constants.cpp (original)
+++ llvm/trunk/lib/VMCore/Constants.cpp Tue Jan 24 20:42:41 2012
@@ -129,7 +129,7 @@
// Broadcast a scalar to a vector, if necessary.
if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- C = ConstantVector::getSplat(VTy->getNumElements(), C);
+ C = ConstantVector::get(std::vector<Constant *>(VTy->getNumElements(), C));
return C;
}
@@ -145,9 +145,11 @@
return ConstantFP::get(Ty->getContext(), FL);
}
+ SmallVector<Constant*, 16> Elts;
VectorType *VTy = cast<VectorType>(Ty);
- return ConstantVector::getSplat(VTy->getNumElements(),
- getAllOnesValue(VTy->getElementType()));
+ Elts.resize(VTy->getNumElements(), getAllOnesValue(VTy->getElementType()));
+ assert(Elts[0] && "Invalid AllOnes value!");
+ return cast<ConstantVector>(ConstantVector::get(Elts));
}
void Constant::destroyConstantImpl() {
@@ -392,8 +394,9 @@
}
assert(VTy->getElementType()->isIntegerTy(1) &&
"True must be vector of i1 or i1.");
- return ConstantVector::getSplat(VTy->getNumElements(),
- ConstantInt::getTrue(Ty->getContext()));
+ SmallVector<Constant*, 16> Splat(VTy->getNumElements(),
+ ConstantInt::getTrue(Ty->getContext()));
+ return ConstantVector::get(Splat);
}
Constant *ConstantInt::getFalse(Type *Ty) {
@@ -404,8 +407,9 @@
}
assert(VTy->getElementType()->isIntegerTy(1) &&
"False must be vector of i1 or i1.");
- return ConstantVector::getSplat(VTy->getNumElements(),
- ConstantInt::getFalse(Ty->getContext()));
+ SmallVector<Constant*, 16> Splat(VTy->getNumElements(),
+ ConstantInt::getFalse(Ty->getContext()));
+ return ConstantVector::get(Splat);
}
@@ -429,7 +433,8 @@
// For vectors, broadcast the value.
if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- return ConstantVector::getSplat(VTy->getNumElements(), C);
+ return ConstantVector::get(SmallVector<Constant*,
+ 16>(VTy->getNumElements(), C));
return C;
}
@@ -454,7 +459,8 @@
// For vectors, broadcast the value.
if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- return ConstantVector::getSplat(VTy->getNumElements(), C);
+ return ConstantVector::get(
+ SmallVector<Constant *, 16>(VTy->getNumElements(), C));
return C;
}
@@ -500,7 +506,8 @@
// For vectors, broadcast the value.
if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- return ConstantVector::getSplat(VTy->getNumElements(), C);
+ return ConstantVector::get(
+ SmallVector<Constant *, 16>(VTy->getNumElements(), C));
return C;
}
@@ -514,7 +521,8 @@
// For vectors, broadcast the value.
if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- return ConstantVector::getSplat(VTy->getNumElements(), C);
+ return ConstantVector::get(
+ SmallVector<Constant *, 16>(VTy->getNumElements(), C));
return C;
}
@@ -529,12 +537,15 @@
Constant *ConstantFP::getZeroValueForNegation(Type* Ty) {
- if (Ty->getScalarType()->isFloatingPointTy()) {
- Constant *C = getNegativeZero(Ty);
- if (VectorType *VTy = dyn_cast<VectorType>(Ty))
- return ConstantVector::getSplat(VTy->getNumElements(), C);
- return C;
- }
+ if (VectorType *PTy = dyn_cast<VectorType>(Ty))
+ if (PTy->getElementType()->isFloatingPointTy()) {
+ SmallVector<Constant*, 16> zeros(PTy->getNumElements(),
+ getNegativeZero(PTy->getElementType()));
+ return ConstantVector::get(zeros);
+ }
+
+ if (Ty->isFloatingPointTy())
+ return getNegativeZero(Ty);
return Constant::getNullValue(Ty);
}
@@ -807,12 +818,6 @@
return pImpl->VectorConstants.getOrCreate(T, V);
}
-Constant *ConstantVector::getSplat(unsigned NumElts, Constant *V) {
- SmallVector<Constant*, 32> Elts(NumElts, V);
- return get(Elts);
-}
-
-
// Utility function for determining if a ConstantExpr is a CastOp or not. This
// can't be inline because we don't want to #include Instruction.h into
// Constant.h
@@ -2189,38 +2194,6 @@
return getImpl(StringRef((char*)Elts.data(), Elts.size()*8), Ty);
}
-Constant *ConstantDataVector::getSplat(unsigned NumElts, Constant *V) {
- assert(isElementTypeCompatible(V->getType()) &&
- "Element type not compatible with ConstantData");
- if (ConstantInt *CI = dyn_cast<ConstantInt>(V)) {
- if (CI->getType()->isIntegerTy(8)) {
- SmallVector<uint8_t, 16> Elts(NumElts, CI->getZExtValue());
- return get(V->getContext(), Elts);
- }
- if (CI->getType()->isIntegerTy(16)) {
- SmallVector<uint16_t, 16> Elts(NumElts, CI->getZExtValue());
- return get(V->getContext(), Elts);
- }
- if (CI->getType()->isIntegerTy(32)) {
- SmallVector<uint32_t, 16> Elts(NumElts, CI->getZExtValue());
- return get(V->getContext(), Elts);
- }
- assert(CI->getType()->isIntegerTy(64) && "Unsupported ConstantData type");
- SmallVector<uint64_t, 16> Elts(NumElts, CI->getZExtValue());
- return get(V->getContext(), Elts);
- }
-
- ConstantFP *CFP = cast<ConstantFP>(V);
- if (CFP->getType()->isFloatTy()) {
- SmallVector<float, 16> Elts(NumElts, CFP->getValueAPF().convertToFloat());
- return get(V->getContext(), Elts);
- }
- assert(CFP->getType()->isDoubleTy() && "Unsupported ConstantData type");
- SmallVector<double, 16> Elts(NumElts, CFP->getValueAPF().convertToDouble());
- return get(V->getContext(), Elts);
-}
-
-
/// getElementAsInteger - If this is a sequential container of integers (of
/// any size), return the specified element in the low bits of a uint64_t.
uint64_t ConstantDataSequential::getElementAsInteger(unsigned Elt) const {
More information about the llvm-commits
mailing list