[modules] Allow a function template definition if we have a pre-existing but not visible definition of the same template.
llvm-svn: 233430
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index 3b9bb84..ce183d0 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -10282,13 +10282,16 @@
if (canRedefineFunction(Definition, getLangOpts()))
return;
- // If we don't have a visible definition of the function, and it's inline,
- // it's OK to form another definition of it.
+ // If we don't have a visible definition of the function, and it's inline or
+ // a template, it's OK to form another definition of it.
//
// FIXME: Should we skip the body of the function and use the old definition
// in this case? That may be necessary for functions that return local types
// through a deduced return type, or instantiate templates with local types.
- if (!hasVisibleDefinition(Definition) && Definition->isInlineSpecified())
+ if (!hasVisibleDefinition(Definition) &&
+ (Definition->isInlineSpecified() ||
+ Definition->getDescribedFunctionTemplate() ||
+ Definition->getNumTemplateParameterLists()))
return;
if (getLangOpts().GNUMode && Definition->isInlineSpecified() &&