[llvm-commits] CVS: llvm/lib/Target/IA64/IA64AsmPrinter.cpp

[Duraid Madina]

Changes in directory llvm/lib/Target/IA64:

IA64AsmPrinter.cpp updated: 1.4 -> 1.5
---
Log message:

* declare local common symbols as such (.lcomm, not merely .common)
* begin great adventure into correct function descriptor materialization 



---
Diffs of the changes:  (+10 -7)

 IA64AsmPrinter.cpp |   17 ++++++++++-------
 1 files changed, 10 insertions(+), 7 deletions(-)


Index: llvm/lib/Target/IA64/IA64AsmPrinter.cpp
diff -u llvm/lib/Target/IA64/IA64AsmPrinter.cpp:1.4 llvm/lib/Target/IA64/IA64AsmPrinter.cpp:1.5
--- llvm/lib/Target/IA64/IA64AsmPrinter.cpp:1.4	Mon Mar 28 09:21:43 2005
+++ llvm/lib/Target/IA64/IA64AsmPrinter.cpp	Thu Mar 31 01:40:24 2005
@@ -112,12 +112,15 @@
           (I->hasLinkOnceLinkage() || I->hasInternalLinkage() ||
            I->hasWeakLinkage() /* FIXME: Verify correct */)) {
         SwitchSection(O, CurSection, ".data");
-        if (I->hasInternalLinkage())
-// FIXME          O << "\t.local " << name << "\n";
-        
-        O << "\t.common " << name << "," << TD.getTypeSize(C->getType())
-          << "," << (1 << Align);
-        O << "\t\t// ";
+        if (I->hasInternalLinkage()) {
+	  O << "\t.lcomm " << name << "," << TD.getTypeSize(C->getType())
+	    << "," << (1 << Align);
+	  O << "\t\t// ";
+	} else {
+	  O << "\t.common " << name << "," << TD.getTypeSize(C->getType())
+	    << "," << (1 << Align);
+	  O << "\t\t// ";
+	}
         WriteAsOperand(O, I, true, true, &M);
         O << "\n";
       } else {
@@ -345,7 +348,7 @@
     bool Needfptr=false; // if we're computing an address @ltoff(X), do
                          // we need to decorate it so it becomes
 			 // @ltoff(@fptr(X)) ?
-    if(F && !isBRCALLinsn && F->isExternal())
+    if(F && !isBRCALLinsn /*&& F->isExternal()*/)
       Needfptr=true;
    
     // if this is the target of a call instruction, we should define