From d5a932b92af122209bfb53fc3198698a4af18371 Mon Sep 17 00:00:00 2001 From: Jim Laskey Date: Thu, 15 Jun 2006 19:37:14 +0000 Subject: [PATCH] Was pointed out that structure alignment and type alignment are not the same thing. Doubles still need to be special cased. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@28806 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/AsmPrinter.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/CodeGen/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter.cpp index c5d6cb206fb..9f00fb2bff2 100644 --- a/lib/CodeGen/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter.cpp @@ -250,6 +250,9 @@ unsigned AsmPrinter::getPreferredAlignmentLog(const GlobalVariable *GV) const { Alignment = Log2_32(GV->getAlignment()); 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. -- 2.11.0