[llvm-commits] [llvm] r78006 - /llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp

Chris Lattner sabre at nondot.org
Mon Aug 3 15:16:58 PDT 2009


Author: lattner
Date: Mon Aug  3 17:16:57 2009
New Revision: 78006

URL: http://llvm.org/viewvc/llvm-project?rev=78006&view=rev
Log:
hoist some common code out of a switch

Modified:
    llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp

Modified: llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp?rev=78006&r1=78005&r2=78006&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp (original)
+++ llvm/trunk/lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp Mon Aug  3 17:16:57 2009
@@ -169,23 +169,22 @@
     DecorateCygMingName(CurrentFnName, F);
 
   SwitchToSection(getObjFileLowering().SectionForGlobal(F, Mang, TM));
+  EmitAlignment(FnAlign, F);
+
   switch (F->getLinkage()) {
   default: llvm_unreachable("Unknown linkage type!");
   case Function::InternalLinkage:  // Symbols default to internal.
   case Function::PrivateLinkage:
   case Function::LinkerPrivateLinkage:
-    EmitAlignment(FnAlign, F);
     break;
   case Function::DLLExportLinkage:
   case Function::ExternalLinkage:
-    EmitAlignment(FnAlign, F);
     O << "\t.globl\t" << CurrentFnName << '\n';
     break;
   case Function::LinkOnceAnyLinkage:
   case Function::LinkOnceODRLinkage:
   case Function::WeakAnyLinkage:
   case Function::WeakODRLinkage:
-    EmitAlignment(FnAlign, F);
     if (Subtarget->isTargetDarwin()) {
       O << "\t.globl\t" << CurrentFnName << '\n';
       O << TAI->getWeakDefDirective() << CurrentFnName << '\n';





More information about the llvm-commits mailing list