Handle visibility printing with all generality. Remove bunch of duplicate code.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@54540 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Target/ARM/ARMAsmPrinter.cpp b/lib/Target/ARM/ARMAsmPrinter.cpp
index 8af79de..89b0a8a 100644
--- a/lib/Target/ARM/ARMAsmPrinter.cpp
+++ b/lib/Target/ARM/ARMAsmPrinter.cpp
@@ -226,14 +226,7 @@
break;
}
- const char *VisibilityDirective = NULL;
- if (F->hasHiddenVisibility())
- VisibilityDirective = TAI->getHiddenDirective();
- else if (F->hasProtectedVisibility())
- VisibilityDirective = TAI->getProtectedDirective();
-
- if (VisibilityDirective)
- O << VisibilityDirective << CurrentFnName << "\n";
+ printVisibility(CurrentFnName, F->getVisibility());
if (AFI->isThumbFunction()) {
EmitAlignment(1, F, AFI->getAlign());
@@ -870,14 +863,7 @@
unsigned Size = TD->getABITypeSize(Type);
unsigned Align = TD->getPreferredAlignmentLog(GVar);
- const char *VisibilityDirective = NULL;
- if (GVar->hasHiddenVisibility())
- VisibilityDirective = TAI->getHiddenDirective();
- else if (GVar->hasProtectedVisibility())
- VisibilityDirective = TAI->getProtectedDirective();
-
- if (VisibilityDirective)
- O << VisibilityDirective << name << "\n";
+ printVisibility(name, GVar->getVisibility());
if (Subtarget->isTargetELF())
O << "\t.type " << name << ",%object\n";
diff --git a/lib/Target/Alpha/AlphaAsmPrinter.cpp b/lib/Target/Alpha/AlphaAsmPrinter.cpp
index 1532047..e592e80 100644
--- a/lib/Target/Alpha/AlphaAsmPrinter.cpp
+++ b/lib/Target/Alpha/AlphaAsmPrinter.cpp
@@ -164,6 +164,8 @@
break;
}
+ printVisibility(CurrentFnName, F->getVisibility());
+
O << "\t.ent " << CurrentFnName << "\n";
O << CurrentFnName << ":\n";
@@ -221,8 +223,7 @@
SwitchToDataSection(SectionName.c_str());
// 1: Check visibility
- if (GVar->hasHiddenVisibility())
- O << TAI->getHiddenDirective() << name << "\n";
+ printVisibility(name, GVar->getVisibility());
// 2: Kind
switch (GVar->getLinkage()) {
diff --git a/lib/Target/IA64/IA64AsmPrinter.cpp b/lib/Target/IA64/IA64AsmPrinter.cpp
index 8ae7d90..6ce530d 100644
--- a/lib/Target/IA64/IA64AsmPrinter.cpp
+++ b/lib/Target/IA64/IA64AsmPrinter.cpp
@@ -142,6 +142,9 @@
// Print out labels for the function.
EmitAlignment(5);
O << "\t.global\t" << CurrentFnName << '\n';
+
+ printVisibility(CurrentFnName, F->getVisibility());
+
O << "\t.type\t" << CurrentFnName << ", @function\n";
O << CurrentFnName << ":\n";
@@ -274,7 +277,7 @@
unsigned Size = TD->getABITypeSize(C->getType());
unsigned Align = TD->getPreferredAlignmentLog(GVar);
- // FIXME: ELF supports visibility
+ printVisibility(name, GVar->getVisibility());
SwitchToDataSection(SectionName.c_str());
diff --git a/lib/Target/Mips/MipsAsmPrinter.cpp b/lib/Target/Mips/MipsAsmPrinter.cpp
index 0d222f1..42979fc 100644
--- a/lib/Target/Mips/MipsAsmPrinter.cpp
+++ b/lib/Target/Mips/MipsAsmPrinter.cpp
@@ -237,6 +237,8 @@
O << "\t.globl\t" << CurrentFnName << '\n';
O << "\t.ent\t" << CurrentFnName << '\n';
+ printVisibility(CurrentFnName, F->getVisibility());
+
if ((TAI->hasDotTypeDotSizeDirective()) && Subtarget->isLinux())
O << "\t.type\t" << CurrentFnName << ", @function\n";
@@ -497,7 +499,7 @@
} else
Align = TD->getPreferredTypeAlignmentShift(CTy);
- // FIXME: ELF supports visibility
+ printVisibility(name, GVar->getVisibility());
SwitchToDataSection(SectionName.c_str());
@@ -508,7 +510,7 @@
if (GVar->hasInternalLinkage())
O << "\t.local\t" << name << '\n';
-
+
O << TAI->getCOMMDirective() << name << ',' << Size;
if (TAI->getCOMMDirectiveTakesAlignment())
O << ',' << (1 << Align);
diff --git a/lib/Target/PowerPC/PPCAsmPrinter.cpp b/lib/Target/PowerPC/PPCAsmPrinter.cpp
index e3186f2..1c54590 100644
--- a/lib/Target/PowerPC/PPCAsmPrinter.cpp
+++ b/lib/Target/PowerPC/PPCAsmPrinter.cpp
@@ -593,9 +593,7 @@
break;
}
- if (F->hasHiddenVisibility())
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << CurrentFnName << '\n';
+ printVisibility(CurrentFnName, F->getVisibility());
EmitAlignment(2, F);
O << CurrentFnName << ":\n";
@@ -671,9 +669,7 @@
std::string name = Mang->getValueName(GVar);
std::string SectionName = TAI->SectionForGlobal(GVar);
- if (GVar->hasHiddenVisibility())
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << name << '\n';
+ printVisibility(name, GVar->getVisibility());
Constant *C = GVar->getInitializer();
const Type *Type = C->getType();
@@ -792,9 +788,7 @@
break;
}
- if (F->hasHiddenVisibility())
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << CurrentFnName << '\n';
+ printVisibility(CurrentFnName, F->getVisibility());
EmitAlignment(OptimizeForSize ? 2 : 4, F);
O << CurrentFnName << ":\n";
@@ -909,9 +903,7 @@
std::string name = Mang->getValueName(GVar);
std::string SectionName = TAI->SectionForGlobal(GVar);
- if (GVar->hasHiddenVisibility())
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << name << '\n';
+ printVisibility(name, GVar->getVisibility());
Constant *C = GVar->getInitializer();
const Type *Type = C->getType();
diff --git a/lib/Target/Sparc/SparcAsmPrinter.cpp b/lib/Target/Sparc/SparcAsmPrinter.cpp
index 2752958..5cdb4d6 100644
--- a/lib/Target/Sparc/SparcAsmPrinter.cpp
+++ b/lib/Target/Sparc/SparcAsmPrinter.cpp
@@ -107,6 +107,9 @@
SwitchToTextSection(getSectionForFunction(*F).c_str(), F);
EmitAlignment(4, F);
O << "\t.globl\t" << CurrentFnName << '\n';
+
+ printVisibility(CurrentFnName, F->getVisibility());
+
O << "\t.type\t" << CurrentFnName << ", #function\n";
O << CurrentFnName << ":\n";
@@ -250,7 +253,8 @@
unsigned Size = TD->getABITypeSize(C->getType());
unsigned Align = TD->getPreferredAlignment(GVar);
- // FIXME: ELF supports visibility
+ printVisibility(name, GVar->getVisibility());
+
SwitchToDataSection(SectionName.c_str());
if (C->isNullValue() && !GVar->hasSection()) {
diff --git a/lib/Target/X86/X86ATTAsmPrinter.cpp b/lib/Target/X86/X86ATTAsmPrinter.cpp
index f99566b..bb51956 100644
--- a/lib/Target/X86/X86ATTAsmPrinter.cpp
+++ b/lib/Target/X86/X86ATTAsmPrinter.cpp
@@ -183,13 +183,8 @@
}
break;
}
- if (F->hasHiddenVisibility()) {
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << CurrentFnName << '\n';
- } else if (F->hasProtectedVisibility()) {
- if (const char *Directive = TAI->getProtectedDirective())
- O << Directive << CurrentFnName << '\n';
- }
+
+ printVisibility(CurrentFnName, F->getVisibility());
if (Subtarget->isTargetELF())
O << "\t.type\t" << CurrentFnName << ",@function\n";
@@ -773,13 +768,7 @@
unsigned Size = TD->getABITypeSize(Type);
unsigned Align = TD->getPreferredAlignmentLog(GVar);
- if (GVar->hasHiddenVisibility()) {
- if (const char *Directive = TAI->getHiddenDirective())
- O << Directive << name << '\n';
- } else if (GVar->hasProtectedVisibility()) {
- if (const char *Directive = TAI->getProtectedDirective())
- O << Directive << name << '\n';
- }
+ printVisibility(name, GVar->getVisibility());
if (Subtarget->isTargetELF())
O << "\t.type\t" << name << ",@object\n";