[llvm-commits] [llvm] r78242 - in /llvm/trunk: include/llvm/Target/TargetAsmInfo.h lib/CodeGen/AsmPrinter/AsmPrinter.cpp lib/Target/TargetAsmInfo.cpp lib/Target/X86/X86TargetAsmInfo.cpp

[Chris Lattner]

Author: lattner
Date: Wed Aug  5 15:49:52 2009
New Revision: 78242

URL: http://llvm.org/viewvc/llvm-project?rev=78242&view=rev
Log:
remove the 'DataSectionStartSuffix' and 'TextSectionStartSuffix' knobs.

Modified:
    llvm/trunk/include/llvm/Target/TargetAsmInfo.h
    llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
    llvm/trunk/lib/Target/TargetAsmInfo.cpp
    llvm/trunk/lib/Target/X86/X86TargetAsmInfo.cpp

Modified: llvm/trunk/include/llvm/Target/TargetAsmInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetAsmInfo.h?rev=78242&r1=78241&r2=78242&view=diff

==============================================================================
--- llvm/trunk/include/llvm/Target/TargetAsmInfo.h (original)
+++ llvm/trunk/include/llvm/Target/TargetAsmInfo.h Wed Aug  5 15:49:52 2009
@@ -190,14 +190,6 @@
     /// this.
     const char *SwitchToSectionDirective; // Defaults to "\t.section\t"
     
-    /// TextSectionStartSuffix - This is printed after each start of section
-    /// directive for text sections.
-    const char *TextSectionStartSuffix;   // Defaults to "".
-
-    /// DataSectionStartSuffix - This is printed after each start of section
-    /// directive for data sections.
-    const char *DataSectionStartSuffix;   // Defaults to "".
-    
     /// JumpTableDirective - if non-null, the directive to emit before a jump
     /// table.
     const char *JumpTableDirective;
@@ -444,12 +436,6 @@
     const char *getSwitchToSectionDirective() const {
       return SwitchToSectionDirective;
     }
-    const char *getTextSectionStartSuffix() const {
-      return TextSectionStartSuffix;
-    }
-    const char *getDataSectionStartSuffix() const {
-      return DataSectionStartSuffix;
-    }
     const char *getGlobalDirective() const {
       return GlobalDirective;
     }

Modified: llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp?rev=78242&r1=78241&r2=78242&view=diff

==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp Wed Aug  5 15:49:52 2009
@@ -92,21 +92,19 @@
 
   CurrentSection = NS;
 
-  if (NS != 0) {
-    // If section is named we need to switch into it via special '.section'
-    // directive and also append funky flags. Otherwise - section name is just
-    // some magic assembler directive.
-    if (!NS->isDirective()) {
-      SmallString<32> FlagsStr;
-      
-      getObjFileLowering().getSectionFlagsAsString(NS->getKind(), FlagsStr);
+  if (NS == 0) return;
+  
+  // If section is named we need to switch into it via special '.section'
+  // directive and also append funky flags. Otherwise - section name is just
+  // some magic assembler directive.
+  if (!NS->isDirective()) {
+    SmallString<32> FlagsStr;
+    getObjFileLowering().getSectionFlagsAsString(NS->getKind(), FlagsStr);
 
-      O << TAI->getSwitchToSectionDirective()
-        << CurrentSection->getName() << FlagsStr.c_str();
-    } else {
-      O << CurrentSection->getName();
-    }
-    O << TAI->getDataSectionStartSuffix() << '\n';
+    O << TAI->getSwitchToSectionDirective()
+      << CurrentSection->getName() << FlagsStr.c_str() << '\n';
+  } else {
+    O << CurrentSection->getName() << '\n';
   }
 }
 

Modified: llvm/trunk/lib/Target/TargetAsmInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/TargetAsmInfo.cpp?rev=78242&r1=78241&r2=78242&view=diff

==============================================================================
--- llvm/trunk/lib/Target/TargetAsmInfo.cpp (original)
+++ llvm/trunk/lib/Target/TargetAsmInfo.cpp Wed Aug  5 15:49:52 2009
@@ -55,8 +55,6 @@
   AlignmentIsInBytes = true;
   TextAlignFillValue = 0;
   SwitchToSectionDirective = "\t.section\t";
-  TextSectionStartSuffix = "";
-  DataSectionStartSuffix = "";
   JumpTableDirective = 0;
   GlobalDirective = "\t.globl\t";
   SetDirective = 0;

Modified: llvm/trunk/lib/Target/X86/X86TargetAsmInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86TargetAsmInfo.cpp?rev=78242&r1=78241&r2=78242&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/X86TargetAsmInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86TargetAsmInfo.cpp Wed Aug  5 15:49:52 2009
@@ -130,10 +130,6 @@
   HasSingleParameterDotFile = false;
 
   AlignmentIsInBytes = true;
-
-  SwitchToSectionDirective = "";
-  TextSectionStartSuffix = "\tSEGMENT PARA 'CODE'";
-  DataSectionStartSuffix = "\tSEGMENT PARA 'DATA'";
 }
 
 // Instantiate default implementation.