From ffc29be83f336e6da110442787c8e1ff0780e0a0 Mon Sep 17 00:00:00 2001 From: Daniel Dunbar Date: Wed, 6 Aug 2008 00:03:29 +0000 Subject: Implement GNU asm-label extension support in CodeGen. This fixes scimark2 on Darwin. - Added Sema support for asm-label on variables, which I forgot before. - Update CodeGen to use GlobalDeclMap to determine if static Decls require emission (instead of LLVM module name lookup). Important since the Decl name and the LLVM module name can differ. - llvm-svn: 54388 --- clang/lib/CodeGen/CodeGenModule.cpp | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) (limited to 'clang/lib/CodeGen/CodeGenModule.cpp') diff --git a/clang/lib/CodeGen/CodeGenModule.cpp b/clang/lib/CodeGen/CodeGenModule.cpp index 176576acbbc2..ebe75dd4a2b7 100644 --- a/clang/lib/CodeGen/CodeGenModule.cpp +++ b/clang/lib/CodeGen/CodeGenModule.cpp @@ -184,6 +184,12 @@ void CodeGenModule::SetGlobalValueAttributes(const FunctionDecl *FD, if (const VisibilityAttr *attr = FD->getAttr()) CodeGenModule::setVisibility(GV, attr->getVisibility()); // FIXME: else handle -fvisibility + + if (const AsmLabelAttr *ALA = FD->getAttr()) { + // Prefaced with special LLVM marker to indicate that the name + // should not be munged. + GV->setName("\01" + ALA->getLabel()); + } } void CodeGenModule::SetFunctionAttributes(const FunctionDecl *FD, @@ -400,13 +406,8 @@ void CodeGenModule::EmitStatics() { // Check if we have used a decl with the same name // FIXME: The AST should have some sort of aggregate decls or // global symbol map. - if (const FunctionDecl *FD = dyn_cast(D)) { - if (!getModule().getFunction(FD->getName())) - continue; - } else { - if (!getModule().getNamedGlobal(cast(D)->getName())) - continue; - } + if (!GlobalDeclMap.count(D->getName())) + continue; // Emit the definition. EmitGlobalDefinition(D); @@ -620,6 +621,12 @@ void CodeGenModule::EmitGlobalVarDefinition(const VarDecl *D) { if (const VisibilityAttr *attr = D->getAttr()) setVisibility(GV, attr->getVisibility()); // FIXME: else handle -fvisibility + + if (const AsmLabelAttr *ALA = D->getAttr()) { + // Prefaced with special LLVM marker to indicate that the name + // should not be munged. + GV->setName("\01" + ALA->getLabel()); + } // Set the llvm linkage type as appropriate. if (D->getStorageClass() == VarDecl::Static) -- cgit v1.2.3