64-bit (MMX) vectors do not need restrictive alignment.

128-bit vectors need it only when SSE is on.



git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@46890 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Dale Johannesen 2008-02-08 19:48:20 +00:00
parent 86383e1f7c
commit 0c191872ab

View File

@ -704,9 +704,6 @@ static void getMaxByValAlign(const Type *Ty, unsigned &MaxAlign) {
if (const VectorType *VTy = dyn_cast<VectorType>(Ty)) { if (const VectorType *VTy = dyn_cast<VectorType>(Ty)) {
if (VTy->getBitWidth() == 128) if (VTy->getBitWidth() == 128)
MaxAlign = 16; MaxAlign = 16;
else if (VTy->getBitWidth() == 64)
if (MaxAlign < 8)
MaxAlign = 8;
} else if (const ArrayType *ATy = dyn_cast<ArrayType>(Ty)) { } else if (const ArrayType *ATy = dyn_cast<ArrayType>(Ty)) {
unsigned EltAlign = 0; unsigned EltAlign = 0;
getMaxByValAlign(ATy->getElementType(), EltAlign); getMaxByValAlign(ATy->getElementType(), EltAlign);
@ -727,13 +724,14 @@ static void getMaxByValAlign(const Type *Ty, unsigned &MaxAlign) {
/// getByValTypeAlignment - Return the desired alignment for ByVal aggregate /// getByValTypeAlignment - Return the desired alignment for ByVal aggregate
/// function arguments in the caller parameter area. For X86, aggregates /// function arguments in the caller parameter area. For X86, aggregates
/// that contains are placed at 16-byte boundaries while the rest are at /// that contain SSE vectors are placed at 16-byte boundaries while the rest
/// 4-byte boundaries. /// are at 4-byte boundaries.
unsigned X86TargetLowering::getByValTypeAlignment(const Type *Ty) const { unsigned X86TargetLowering::getByValTypeAlignment(const Type *Ty) const {
if (Subtarget->is64Bit()) if (Subtarget->is64Bit())
return getTargetData()->getABITypeAlignment(Ty); return getTargetData()->getABITypeAlignment(Ty);
unsigned Align = 4; unsigned Align = 4;
getMaxByValAlign(Ty, Align); if (Subtarget->hasSSE1())
getMaxByValAlign(Ty, Align);
return Align; return Align;
} }