- getTypeAlignmentShift() should be returning preferred alignment, not ABI
authorEvan Cheng <evan.cheng@apple.com>
Mon, 22 Jan 2007 23:08:19 +0000 (23:08 +0000)
committerEvan Cheng <evan.cheng@apple.com>
Mon, 22 Jan 2007 23:08:19 +0000 (23:08 +0000)
alignment.
- getPreferredAlignmentLog(): remove Double special case.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@33445 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/TargetData.cpp

index f50e6c3bd398e44de99399b41feb90b3b8aee15f..e9580094949c471341fc2b649945c1e2f4a49915 100644 (file)
@@ -451,7 +451,7 @@ unsigned char TargetData::getTypeAlignmentPref(const Type *Ty) const {
 }
 
 unsigned char TargetData::getTypeAlignmentShift(const Type *Ty) const {
-  unsigned Align = getTypeAlignmentABI(Ty);
+  unsigned Align = getTypeAlignmentPref(Ty);
   assert(!(Align & (Align-1)) && "Alignment is not a power of two!");
   return Log2_32(Align);
 }
@@ -514,8 +514,6 @@ unsigned TargetData::getPreferredAlignmentLog(const GlobalVariable *GV) const {
   
   if (GV->hasInitializer()) {
     // Always round up alignment of global doubles to 8 bytes.
-    if (GV->getType()->getElementType() == Type::DoubleTy && Alignment < 3)
-      Alignment = 3;
     if (Alignment < 4) {
       // If the global is not external, see if it is large.  If so, give it a
       // larger alignment.