From ec4ad5ac9bfec9fcec7d7932a089411fb3245902 Mon Sep 17 00:00:00 2001 From: Richard Smith Date: Tue, 15 Sep 2015 18:51:56 +0000 Subject: [PATCH] [modules] Make sure we make hidden UsingShadowDecls visible to redeclaration lookup for the UsingShadowDecls themselves. llvm-svn: 247714 --- clang/include/clang/Sema/Lookup.h | 2 +- clang/test/Modules/Inputs/using-decl-a.h | 1 + clang/test/Modules/Inputs/using-decl-b.h | 1 + clang/test/Modules/using-decl.cpp | 6 ++++++ 4 files changed, 9 insertions(+), 1 deletion(-) diff --git a/clang/include/clang/Sema/Lookup.h b/clang/include/clang/Sema/Lookup.h index ed447ba14975..1162a1d65abc 100644 --- a/clang/include/clang/Sema/Lookup.h +++ b/clang/include/clang/Sema/Lookup.h @@ -240,7 +240,7 @@ public: // If a using-shadow declaration is hidden, it's never visible, not // even to redeclaration lookup. // FIXME: Should this apply to typedefs and namespace aliases too? - if (isa(ND)) + if (isa(ND) && LookupKind != Sema::LookupUsingDeclName) return false; return (AllowHidden && (AllowHiddenInternal || ND->isExternallyVisible())) || diff --git a/clang/test/Modules/Inputs/using-decl-a.h b/clang/test/Modules/Inputs/using-decl-a.h index 85a4788e7647..1d1ffe9ed350 100644 --- a/clang/test/Modules/Inputs/using-decl-a.h +++ b/clang/test/Modules/Inputs/using-decl-a.h @@ -1,5 +1,6 @@ typedef int using_decl_type; int using_decl_var; +int merged; namespace UsingDecl { using ::using_decl_type; diff --git a/clang/test/Modules/Inputs/using-decl-b.h b/clang/test/Modules/Inputs/using-decl-b.h index b4f206fa4431..7c03d09960ed 100644 --- a/clang/test/Modules/Inputs/using-decl-b.h +++ b/clang/test/Modules/Inputs/using-decl-b.h @@ -8,6 +8,7 @@ namespace UsingDecl { namespace UsingDecl { using ::using_decl_type; using ::using_decl_var; + using ::merged; } namespace X { diff --git a/clang/test/Modules/using-decl.cpp b/clang/test/Modules/using-decl.cpp index f01688498c55..1677585ddafc 100644 --- a/clang/test/Modules/using-decl.cpp +++ b/clang/test/Modules/using-decl.cpp @@ -3,9 +3,15 @@ // RUN: %clang_cc1 -x objective-c++ -std=c++11 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t -I %S/Inputs %s -verify -UEARLY_IMPORT // RUN: %clang_cc1 -x objective-c++ -std=c++11 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t -I %S/Inputs %s -verify -DEARLY_IMPORT -fno-modules-hide-internal-linkage // RUN: %clang_cc1 -x objective-c++ -std=c++11 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t -I %S/Inputs %s -verify -UEARLY_IMPORT -fno-modules-hide-internal-linkage +// RUN: %clang_cc1 -x objective-c++ -std=c++11 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t -I %S/Inputs %s -verify -DEARLY_IMPORT -fmodules-local-submodule-visibility +// RUN: %clang_cc1 -x objective-c++ -std=c++11 -fmodules -fimplicit-module-maps -fmodules-cache-path=%t -I %S/Inputs %s -verify -UEARLY_IMPORT -fmodules-local-submodule-visibility #ifdef EARLY_IMPORT @import using_decl.a; +namespace UsingDecl { + using ::merged; +} +int k = UsingDecl::merged; #endif namespace Y {