diff options
author | Chris Lattner <sabre@nondot.org> | 2009-07-14 18:17:16 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2009-07-14 18:17:16 +0000 |
commit | b8158acc23f5f0bf235fb1c6a8182a38ec9b00b2 (patch) | |
tree | 83b8562d6775f42cf93a08cf76d96a5107b67747 /lib/Target/IA64 | |
parent | 95cf30c444707634bbd950f13405b6c8bcfe496b (diff) | |
download | external_llvm-b8158acc23f5f0bf235fb1c6a8182a38ec9b00b2.zip external_llvm-b8158acc23f5f0bf235fb1c6a8182a38ec9b00b2.tar.gz external_llvm-b8158acc23f5f0bf235fb1c6a8182a38ec9b00b2.tar.bz2 |
Reapply my previous asmprinter changes now with more testing and two
additional bug fixes:
1. The bug that everyone hit was a problem in the asmprinter where it
would remove $stub but keep the L prefix on a name when emitting the
indirect symbol. This is easy to fix by keeping the name of the stub
and the name of the symbol in a StringMap instead of just keeping a
StringSet and trying to reconstruct it late.
2. There was a problem printing the personality function. The current
logic to print out the personality function from the DWARF information
is a bit of a cesspool right now that duplicates a bunch of other
logic in the asm printer. The short version of it is that it depends
on emitting both the L and _ prefix for symbols (at least on darwin)
and until I can untangle it, it is best to switch the mangler back to
emitting both prefixes.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@75646 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/IA64')
-rw-r--r-- | lib/Target/IA64/AsmPrinter/IA64AsmPrinter.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/Target/IA64/AsmPrinter/IA64AsmPrinter.cpp b/lib/Target/IA64/AsmPrinter/IA64AsmPrinter.cpp index e7ed64e..6797868 100644 --- a/lib/Target/IA64/AsmPrinter/IA64AsmPrinter.cpp +++ b/lib/Target/IA64/AsmPrinter/IA64AsmPrinter.cpp @@ -201,16 +201,16 @@ void IA64AsmPrinter::printOp(const MachineOperand &MO, // Intel ias rightly complains of an 'undefined symbol') if (F /*&& isBRCALLinsn*/ && F->isDeclaration()) - ExternalFunctionNames.insert(Mang->getValueName(MO.getGlobal())); + ExternalFunctionNames.insert(Mang->getMangledName(MO.getGlobal())); else if (GV->isDeclaration()) // e.g. stuff like 'stdin' - ExternalObjectNames.insert(Mang->getValueName(MO.getGlobal())); + ExternalObjectNames.insert(Mang->getMangledName(MO.getGlobal())); if (!isBRCALLinsn) O << "@ltoff("; if (Needfptr) O << "@fptr("; - O << Mang->getValueName(MO.getGlobal()); + O << Mang->getMangledName(MO.getGlobal()); if (Needfptr && !isBRCALLinsn) O << "#))"; // close both fptr( and ltoff( @@ -268,7 +268,7 @@ void IA64AsmPrinter::printModuleLevelGV(const GlobalVariable* GVar) { return; O << "\n\n"; - std::string name = Mang->getValueName(GVar); + std::string name = Mang->getMangledName(GVar); Constant *C = GVar->getInitializer(); if (isa<MDNode>(C) || isa<MDString>(C)) return; |