[llvm-commits] [llvm] r52071 - /llvm/trunk/lib/Target/CBackend/CBackend.cpp

[Evan Cheng]

Author: evancheng
Date: Sat Jun  7 02:50:29 2008
New Revision: 52071

URL: http://llvm.org/viewvc/llvm-project?rev=52071&view=rev
Log:
Revert r52046. It broke cbe on x86 / Mac OS X.

Modified:
    llvm/trunk/lib/Target/CBackend/CBackend.cpp

Modified: llvm/trunk/lib/Target/CBackend/CBackend.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/CBackend/CBackend.cpp?rev=52071&r1=52070&r2=52071&view=diff

==============================================================================
--- llvm/trunk/lib/Target/CBackend/CBackend.cpp (original)
+++ llvm/trunk/lib/Target/CBackend/CBackend.cpp Sat Jun  7 02:50:29 2008
@@ -1224,10 +1224,6 @@
     Name = "llvm_cbe_" + VarName;
   } else {
     Name = Mang->getValueName(Operand);
-
-    // Check, if operand has assembler identifier and handle it separately
-    if (Operand->getNameStart()[0] == 1)
-      Name = "llvm_cbe_asmname_" + Name;
   }
 
   return Name;
@@ -1656,11 +1652,6 @@
 
       if (I->hasExternalWeakLinkage())
          Out << " __EXTERNAL_WEAK__";
-
-      // Special handling for assembler identifiers
-      if (I->getNameStart()[0] == 1)
-        Out << " LLVM_ASM(\"" << I->getName().c_str()+1 << "\")";
-
       Out << ";\n";
     }
   }
@@ -1670,7 +1661,7 @@
   Out << "double fmod(double, double);\n";   // Support for FP rem
   Out << "float fmodf(float, float);\n";
   Out << "long double fmodl(long double, long double);\n";
-
+  
   for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) {
     // Don't print declarations for intrinsic functions.
     if (!I->isIntrinsic() && I->getName() != "setjmp" &&
@@ -1678,7 +1669,7 @@
       if (I->hasExternalWeakLinkage())
         Out << "extern ";
       printFunctionSignature(I, true);
-      if (I->hasWeakLinkage() || I->hasLinkOnceLinkage())
+      if (I->hasWeakLinkage() || I->hasLinkOnceLinkage()) 
         Out << " __ATTRIBUTE_WEAK__";
       if (I->hasExternalWeakLinkage())
         Out << " __EXTERNAL_WEAK__";
@@ -1688,11 +1679,10 @@
         Out << " __ATTRIBUTE_DTOR__";
       if (I->hasHiddenVisibility())
         Out << " __HIDDEN__";
-
-      // Special handling for assembler identifiers
-      if (I->getNameStart()[0] == 1)
+      
+      if (I->hasName() && I->getName()[0] == 1)
         Out << " LLVM_ASM(\"" << I->getName().c_str()+1 << "\")";
-
+          
       Out << ";\n";
     }
   }
@@ -1729,11 +1719,6 @@
           Out << " __EXTERNAL_WEAK__";
         if (I->hasHiddenVisibility())
           Out << " __HIDDEN__";
-
-        // Special handling for assembler identifiers
-        if (I->getNameStart()[0] == 1)
-          Out << " LLVM_ASM(\"" << I->getName().c_str()+1 << "\")";
-
         Out << ";\n";
       }
   }
@@ -1741,7 +1726,7 @@
   // Output the global variable definitions and contents...
   if (!M.global_empty()) {
     Out << "\n\n/* Global Variable Definitions and Initialization */\n";
-    for (Module::global_iterator I = M.global_begin(), E = M.global_end();
+    for (Module::global_iterator I = M.global_begin(), E = M.global_end(); 
          I != E; ++I)
       if (!I->isDeclaration()) {
         // Ignore special globals, such as debug info.