From ad5e9cac02c5b5e311d5f768eeb801c053d3c867 Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Wed, 25 Jul 2007 23:36:05 +0000 Subject: [PATCH] Mac OS X should use 0x90 to fill in gaps to satisfy function alignment requirements. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@40501 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/X86/X86ATTAsmPrinter.cpp | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/lib/Target/X86/X86ATTAsmPrinter.cpp b/lib/Target/X86/X86ATTAsmPrinter.cpp index 07670ea62be..4c707df1e39 100644 --- a/lib/Target/X86/X86ATTAsmPrinter.cpp +++ b/lib/Target/X86/X86ATTAsmPrinter.cpp @@ -99,19 +99,28 @@ bool X86ATTAsmPrinter::runOnMachineFunction(MachineFunction &MF) { switch (F->getLinkage()) { default: assert(0 && "Unknown linkage type!"); case Function::InternalLinkage: // Symbols default to internal. - EmitAlignment(4, F); // FIXME: This should be parameterized somewhere. + if (Subtarget->isTargetDarwin()) + // FIXME: This should be parameterized somewhere. + EmitAlignment(4, F, 0, true, 0x90); + else + EmitAlignment(4, F); // FIXME: This should be parameterized somewhere. break; case Function::DLLExportLinkage: DLLExportedFns.insert(Mang->makeNameProper(F->getName(), "")); //FALLS THROUGH case Function::ExternalLinkage: - EmitAlignment(4, F); // FIXME: This should be parameterized somewhere. + if (Subtarget->isTargetDarwin()) + // FIXME: This should be parameterized somewhere. + EmitAlignment(4, F, 0, true, 0x90); + else + EmitAlignment(4, F); // FIXME: This should be parameterized somewhere. O << "\t.globl\t" << CurrentFnName << "\n"; break; case Function::LinkOnceLinkage: case Function::WeakLinkage: if (Subtarget->isTargetDarwin()) { - EmitAlignment(4, F); // FIXME: This should be parameterized somewhere. + // FIXME: This should be parameterized somewhere. + EmitAlignment(4, F, 0, true, 0x90); O << "\t.globl\t" << CurrentFnName << "\n"; O << "\t.weak_definition\t" << CurrentFnName << "\n"; } else if (Subtarget->isTargetCygMing()) {