From: Rafael Espindola Date: Mon, 21 May 2012 20:15:56 +0000 (+0000) Subject: Produce a hidden symbol for zed in X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=0b0ad0abd8518742c57d0c6fc01c79982ba2bfdf;p=clang Produce a hidden symbol for zed in struct HIDDEN foo { }; template struct bar { }; template <> struct HIDDEN bar { DEFAULT static void zed(); }; void bar::zed() { } Before we would produce a hidden symbol in struct HIDDEN foo { }; template struct bar { }; template <> struct bar { DEFAULT static void zed(); }; void bar::zed() { } But adding HIDDEN to the specialization would cause us to produce a default symbol. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@157206 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/AST/Decl.cpp b/lib/AST/Decl.cpp index d31cf0f010..8bc9eb396e 100644 --- a/lib/AST/Decl.cpp +++ b/lib/AST/Decl.cpp @@ -163,7 +163,7 @@ static bool shouldConsiderTemplateLV(const FunctionDecl *fn) { } static bool shouldConsiderTemplateLV(const ClassTemplateSpecializationDecl *d) { - return !d->hasAttr(); + return !d->hasAttr() || d->isExplicitSpecialization(); } static LinkageInfo getLVForNamespaceScopeDecl(const NamedDecl *D, diff --git a/test/CodeGenCXX/visibility.cpp b/test/CodeGenCXX/visibility.cpp index 568a2859f8..be4348ee93 100644 --- a/test/CodeGenCXX/visibility.cpp +++ b/test/CodeGenCXX/visibility.cpp @@ -789,3 +789,19 @@ namespace test39 { // GCC produces a default for this one. Why? // CHECK-HIDDEN: define weak_odr hidden void @_ZN6test391AINS_8hidden_tEE1BIS1_E4tempIS1_EEvv } + +namespace test42 { + struct HIDDEN foo { + }; + template + struct bar { + }; + template <> + struct HIDDEN bar { + DEFAULT static void zed(); + }; + void bar::zed() { + } + // CHECK: define hidden void @_ZN6test423barINS_3fooEE3zedEv + // CHECK-HIDDEN: define hidden void @_ZN6test423barINS_3fooEE3zedEv +}