Index: clang/lib/AST/Decl.cpp =================================================================== --- clang/lib/AST/Decl.cpp +++ clang/lib/AST/Decl.cpp @@ -391,11 +391,18 @@ bool considerVisibility = shouldConsiderTemplateVisibility(fn, specInfo); - // Merge information from the template parameters. FunctionTemplateDecl *temp = specInfo->getTemplate(); - LinkageInfo tempLV = - getLVForTemplateParameterList(temp->getTemplateParameters(), computation); - LV.mergeMaybeWithVisibility(tempLV, considerVisibility); + + // Merge information from the template declaration. + LinkageInfo tempLV = getLVForDecl(temp, computation); + // The linkage of the specialization should be consistent with the + // template declaration. + LV.setLinkage(tempLV.getLinkage()); + + // Merge information from the template parameters. + LinkageInfo paramsLV = + getLVForTemplateParameterList(temp->getTemplateParameters(), computation); + LV.mergeMaybeWithVisibility(paramsLV, considerVisibility); // Merge information from the template arguments. const TemplateArgumentList &templateArgs = *specInfo->TemplateArguments; Index: clang/test/CodeGenCXX/inconsistent-export-template.cpp =================================================================== --- /dev/null +++ clang/test/CodeGenCXX/inconsistent-export-template.cpp @@ -0,0 +1,11 @@ +// RUN: %clang_cc1 -std=c++20 %s -S -emit-llvm -triple %itanium_abi_triple -disable-llvm-passes -o - | FileCheck %s + +export module m; +export template +void f() { +} + +// CHECK: void @_Z1fIiEvv +template <> +void f() { +} Index: clang/test/Modules/inconsist-export-template.cpp =================================================================== --- /dev/null +++ clang/test/Modules/inconsist-export-template.cpp @@ -0,0 +1,33 @@ +// RUN: %clang_cc1 -std=c++20 %s -fsyntax-only -verify +// expected-no-diagnostics +export module m; +export template +void f() { +} + +template <> +void f() { +} + +template +void f1() { +} + +// FIXME: We should reject following specialization, +// since it tries to export a name which is already introduced. +// See https://llvm.org/PR54189 for details. +export template <> +void f1() { +} + +export template +class C { +}; + +template <> +class C { +public: + void M(){}; +}; + +void Use(C &p) { p.M(); }