commit 07c874547bd7fa6ccf0e9d9419b0ebf197f8d6fc
author Vikram S. Adve <vadve@cs.uiuc.edu> Sun Mar 31 19:03:58 2002 +0000
committer Vikram S. Adve <vadve@cs.uiuc.edu> Sun Mar 31 19:03:58 2002 +0000
tree a80cbbbb2302e95db1aab559f2eebdabe9be8f78
parent 429621cad000461601feab3b2204bf1159f85a6b

Minor cleanup in printing constants.  I think this included a bug
fix related to putting a read-write variable in a read-only section,
but I'm not sure now.

llvm-svn: 2073

llvm/lib/Target/Sparc/EmitAssembly.cpp

diff --git a/llvm/lib/Target/Sparc/EmitAssembly.cpp b/llvm/lib/Target/Sparc/EmitAssembly.cpp
index ae61b8d..9157044 100644
--- a/llvm/lib/Target/Sparc/EmitAssembly.cpp
+++ b/llvm/lib/Target/Sparc/EmitAssembly.cpp
@@ -303,7 +303,7 @@
       {
         int RegNum = (int)op.getAllocatedRegNum();
         
-        // ****this code is temporary till NULL Values are fixed
+        // better to print code with NULL registers than to die
         if (RegNum == Target.getRegInfo().getInvalidRegNum()) {
           toAsm << "<NULL VALUE>";
         } else {
@@ -732,47 +732,29 @@
     
   // Now, emit the three data sections separately; the cost of I/O should
   // make up for the cost of extra passes over the globals list!
-  // 
-  // Read-only data section (implies initialized)
+  
+  // Section 1 : Read-only data section (implies initialized)
+  enterSection(AsmPrinter::ReadOnlyData);
   for (Module::const_giterator GI=M->gbegin(), GE=M->gend(); GI != GE; ++GI)
-    {
-      const GlobalVariable* GV = *GI;
-      if (GV->hasInitializer() && GV->isConstant())
-        {
-          if (GI == M->gbegin())
-            enterSection(AsmPrinter::ReadOnlyData);
-          printGlobalVariable(GV);
-        }
-    }
+    if ((*GI)->hasInitializer() && (*GI)->isConstant())
+      printGlobalVariable(*GI);
   
   for (std::hash_set<const Constant*>::const_iterator
          I = moduleConstants.begin(),
          E = moduleConstants.end();  I != E; ++I)
     printConstant(*I);
   
-  // Initialized read-write data section
+  // Section 2 : Initialized read-write data section
+  enterSection(AsmPrinter::InitRWData);
   for (Module::const_giterator GI=M->gbegin(), GE=M->gend(); GI != GE; ++GI)
-    {
-      const GlobalVariable* GV = *GI;
-      if (GV->hasInitializer() && ! GV->isConstant())
-        {
-          if (GI == M->gbegin())
-            enterSection(AsmPrinter::InitRWData);
-          printGlobalVariable(GV);
-        }
-    }
+    if ((*GI)->hasInitializer() && ! (*GI)->isConstant())
+      printGlobalVariable(*GI);
   
-  // Uninitialized read-write data section
+  // Section 3 : Uninitialized read-write data section
+  enterSection(AsmPrinter::UninitRWData);
   for (Module::const_giterator GI=M->gbegin(), GE=M->gend(); GI != GE; ++GI)
-    {
-      const GlobalVariable* GV = *GI;
-      if (! GV->hasInitializer())
-        {
-          if (GI == M->gbegin())
-            enterSection(AsmPrinter::UninitRWData);
-          printGlobalVariable(GV);
-        }
-    }
+    if (! (*GI)->hasInitializer())
+      printGlobalVariable(*GI);
   
   toAsm << "\n";
 }