From 84397472364ac3ce3b5dbcc1e6aa93e9a584c32f Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Tue, 5 Apr 2011 15:51:32 +0000 Subject: [PATCH] Print visibility info for external variables. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@128887 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/AsmPrinter/AsmPrinter.cpp | 22 ++++++++++++---------- test/CodeGen/X86/visibility.ll | 7 +++++-- 2 files changed, 17 insertions(+), 12 deletions(-) diff --git a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp index 358a60b355c..3d3abc2fed2 100644 --- a/lib/CodeGen/AsmPrinter/AsmPrinter.cpp +++ b/lib/CodeGen/AsmPrinter/AsmPrinter.cpp @@ -253,22 +253,24 @@ void AsmPrinter::EmitLinkage(unsigned Linkage, MCSymbol *GVSym) const { /// EmitGlobalVariable - Emit the specified global variable to the .s file. void AsmPrinter::EmitGlobalVariable(const GlobalVariable *GV) { - if (!GV->hasInitializer()) // External globals require no code. - return; - - // Check to see if this is a special global used by LLVM, if so, emit it. - if (EmitSpecialLLVMGlobal(GV)) - return; + if (GV->hasInitializer()) { + // Check to see if this is a special global used by LLVM, if so, emit it. + if (EmitSpecialLLVMGlobal(GV)) + return; - if (isVerbose()) { - WriteAsOperand(OutStreamer.GetCommentOS(), GV, - /*PrintType=*/false, GV->getParent()); - OutStreamer.GetCommentOS() << '\n'; + if (isVerbose()) { + WriteAsOperand(OutStreamer.GetCommentOS(), GV, + /*PrintType=*/false, GV->getParent()); + OutStreamer.GetCommentOS() << '\n'; + } } MCSymbol *GVSym = Mang->getSymbol(GV); EmitVisibility(GVSym, GV->getVisibility()); + if (!GV->hasInitializer()) // External globals require no extra code. + return; + if (MAI->hasDotTypeDotSizeDirective()) OutStreamer.EmitSymbolAttribute(GVSym, MCSA_ELF_TypeObject); diff --git a/test/CodeGen/X86/visibility.ll b/test/CodeGen/X86/visibility.ll index a8d287083a8..580c3dc9266 100644 --- a/test/CodeGen/X86/visibility.ll +++ b/test/CodeGen/X86/visibility.ll @@ -1,11 +1,14 @@ ; RUN: llc -mtriple=x86_64-unknown-linux-gnu %s -o - | FileCheck %s +@zed = external hidden constant i32 + define hidden void @foo() nounwind { entry: - call void @bar() + call void @bar(i32* @zed) ret void } -declare hidden void @bar() +declare hidden void @bar(i32*) +;CHECK: .hidden zed ;CHECK: .hidden bar -- 2.11.0