Alignment of globals has not been quite right. Needed to drop the pointer type
to get the alignment of the element type. llvm-svn: 28799
This commit is contained in:
		
							parent
							
								
									66f0e09313
								
							
						
					
					
						commit
						dce07568f2
					
				| 
						 | 
					@ -244,18 +244,16 @@ void AsmPrinter::EmitXXStructorList(Constant *List) {
 | 
				
			||||||
/// specified global, returned in log form.  This includes an explicitly
 | 
					/// specified global, returned in log form.  This includes an explicitly
 | 
				
			||||||
/// requested alignment (if the global has one).
 | 
					/// requested alignment (if the global has one).
 | 
				
			||||||
unsigned AsmPrinter::getPreferredAlignmentLog(const GlobalVariable *GV) const {
 | 
					unsigned AsmPrinter::getPreferredAlignmentLog(const GlobalVariable *GV) const {
 | 
				
			||||||
  unsigned Alignment = TM.getTargetData()->getTypeAlignmentShift(GV->getType());
 | 
					  const Type *ElemType = GV->getType()->getElementType();
 | 
				
			||||||
 | 
					  unsigned Alignment = TM.getTargetData()->getTypeAlignmentShift(ElemType);
 | 
				
			||||||
  if (GV->getAlignment() > (1U << Alignment))
 | 
					  if (GV->getAlignment() > (1U << Alignment))
 | 
				
			||||||
    Alignment = Log2_32(GV->getAlignment());
 | 
					    Alignment = Log2_32(GV->getAlignment());
 | 
				
			||||||
  
 | 
					  
 | 
				
			||||||
  if (GV->hasInitializer()) {
 | 
					  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 (Alignment < 4) {
 | 
				
			||||||
      // If the global is not external, see if it is large.  If so, give it a
 | 
					      // If the global is not external, see if it is large.  If so, give it a
 | 
				
			||||||
      // larger alignment.
 | 
					      // larger alignment.
 | 
				
			||||||
      if (TM.getTargetData()->getTypeSize(GV->getType()->getElementType()) > 128)
 | 
					      if (TM.getTargetData()->getTypeSize(ElemType) > 128)
 | 
				
			||||||
        Alignment = 4;    // 16-byte alignment.
 | 
					        Alignment = 4;    // 16-byte alignment.
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue