From ec212fb66de0f35f3ecadc4a127ab6bea36bd0bb Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Tue, 7 Feb 2006 21:54:08 +0000 Subject: [PATCH] For ELF, .comm takes alignment value as the optional 3rd argument. It must be specified in bytes. llvm-svn: 26043 --- llvm/lib/Target/X86/X86AsmPrinter.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/llvm/lib/Target/X86/X86AsmPrinter.cpp b/llvm/lib/Target/X86/X86AsmPrinter.cpp index dd19d9fd2a0c..1b05691a0be1 100644 --- a/llvm/lib/Target/X86/X86AsmPrinter.cpp +++ b/llvm/lib/Target/X86/X86AsmPrinter.cpp @@ -105,7 +105,9 @@ bool X86SharedAsmPrinter::doFinalization(Module &M) { O << COMMDirective << name << "," << Size; } else { SwitchSection(".local", I); - O << COMMDirective << name << "," << Size << "," << Align; + O << COMMDirective << name << "," << Size; + if (COMMDirectiveTakesAlignment) + O << "," << (AlignmentIsInBytes ? (1 << Align) : Align); } O << "\t\t" << CommentString << " '" << I->getName() << "'\n"; } else {