Index: cfe/trunk/lib/Sema/SemaLookup.cpp =================================================================== --- cfe/trunk/lib/Sema/SemaLookup.cpp +++ cfe/trunk/lib/Sema/SemaLookup.cpp @@ -1543,8 +1543,21 @@ // and in C we must not because each declaration of a function gets its own // set of declarations for tags in prototype scope. bool VisibleWithinParent; - if (D->isTemplateParameter() || isa(D) || - (isa(DC) && !SemaRef.getLangOpts().CPlusPlus)) + if (D->isTemplateParameter()) { + bool SearchDefinitions = true; + if (const auto *DCD = dyn_cast(DC)) { + if (const auto *TD = DCD->getDescribedTemplate()) { + TemplateParameterList *TPL = TD->getTemplateParameters(); + auto Index = getDepthAndIndex(D).second; + SearchDefinitions = Index >= TPL->size() || TPL->getParam(Index) != D; + } + } + if (SearchDefinitions) + VisibleWithinParent = SemaRef.hasVisibleDefinition(cast(DC)); + else + VisibleWithinParent = isVisible(SemaRef, cast(DC)); + } else if (isa(D) || + (isa(DC) && !SemaRef.getLangOpts().CPlusPlus)) VisibleWithinParent = isVisible(SemaRef, cast(DC)); else if (D->isModulePrivate()) { // A module-private declaration is only visible if an enclosing lexical Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias.h @@ -0,0 +1,7 @@ +#ifndef ALIAS_H +#define ALIAS_H +struct alias_outer { + template + using alias = int; +}; +#endif Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias1.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias1.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias1.h @@ -0,0 +1 @@ +#include "alias.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias2.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias2.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/alias2.h @@ -0,0 +1 @@ +#include "alias.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func.h @@ -0,0 +1,7 @@ +#ifndef FUNC_H +#define FUNC_H +struct func_outer { + template + void func(); +}; +#endif Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func1.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func1.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func1.h @@ -0,0 +1 @@ +#include "func.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func2.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func2.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/func2.h @@ -0,0 +1 @@ +#include "func.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/module.modulemap =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/module.modulemap +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/module.modulemap @@ -0,0 +1,24 @@ +module ALIAS1 { + header "alias1.h" + module ALIAS2 { + header "alias2.h" + } +} +module VAR1 { + header "var1.h" + module VAR2 { + header "var2.h" + } +} +module FUNC1 { + header "func1.h" + module FUNC2 { + header "func2.h" + } +} +module STRCT1 { + header "strct1.h" + module STRCT2 { + header "strct2.h" + } +} Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct.h @@ -0,0 +1,7 @@ +#ifndef STRCT_H +#define STRCT_H +struct strct_outer { + template + struct strct; +}; +#endif Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct1.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct1.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct1.h @@ -0,0 +1 @@ +#include "strct.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct2.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct2.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/strct2.h @@ -0,0 +1 @@ +#include "strct.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var.h @@ -0,0 +1,9 @@ +#ifndef VAR_H +#define VAR_H +struct var_outer { + template + static int var; +}; +#endif + + Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var1.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var1.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var1.h @@ -0,0 +1 @@ +#include "var.h" Index: cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var2.h =================================================================== --- cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var2.h +++ cfe/trunk/test/Modules/Inputs/nested-template-default-arg-redecl/var2.h @@ -0,0 +1 @@ +#include "var.h" Index: cfe/trunk/test/Modules/nested-template-default-arg-redecl.cpp =================================================================== --- cfe/trunk/test/Modules/nested-template-default-arg-redecl.cpp +++ cfe/trunk/test/Modules/nested-template-default-arg-redecl.cpp @@ -0,0 +1,18 @@ +// RUN: rm -rf %t +// RUN: %clang_cc1 -x c++ -fmodules -fimplicit-module-maps -fmodules-cache-path=%t \ +// RUN: -I %S/Inputs/nested-template-default-arg-redecl -std=c++14 \ +// RUN: -fmodules-local-submodule-visibility -verify %s +#include "alias2.h" +#include "var2.h" +#include "strct2.h" +#include "func2.h" + +// FIXME: Variable templates lexical decl context appears to be the translation +// unit, which is incorrect. Fixing this will hopefully address the following +// error/bug: + +// expected-note@Inputs/nested-template-default-arg-redecl/var.h:4 {{default argument declared here}} +auto var = &var_outer::var<>; // expected-error {{default argument of 'var' must be imported from module 'VAR1' before it is required}} +auto func = &func_outer::func<>; +strct_outer::strct<> *strct; +alias_outer::alias<> *alias;