Revert r120063, it was wrong.

llvm-svn: 120296

2 files changed, 5 insertions, 14 deletions

diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp
index 648e654fd7d..d30faccf24a 100644
--- a/clang/lib/Sema/SemaDeclCXX.cpp
+++ b/clang/lib/Sema/SemaDeclCXX.cpp
@@ -3961,8 +3961,8 @@ bool Sema::CheckUsingDeclRedeclaration(SourceLocation UsingLoc,
   //   repeatedly where (and only where) multiple declarations are
   //   allowed.
   //
-  // That's in file contexts.
-  if (CurContext->isFileContext())
+  // That's in non-member contexts.
+  if (!CurContext->getRedeclContext()->isRecord())
     return false;
 
   NestedNameSpecifier *Qual
diff --git a/clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p8.cpp b/clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p8.cpp
index 5ba22c84191..466097171c8 100644
--- a/clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p8.cpp
+++ b/clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p8.cpp
@@ -82,7 +82,7 @@ namespace test2 {
   template struct Derived<int>; // expected-note {{in instantiation of template class}}
 }
 
-// PR8668: redeclarations are not okay in a function.
+// Redeclarations are okay in a function.
 namespace test3 {
   namespace N {
     int f(int);
@@ -90,18 +90,9 @@ namespace test3 {
   }
 
   void g() {
-    using N::f; // expected-note {{previous using declaration}}
-    using N::f; // expected-error {{redeclaration of using decl}}
-    using N::type; // expected-note {{previous using declaration}}
-    using N::type; // expected-error {{redeclaration of using decl}}
-  }
-
-  void h() {
     using N::f;
+    using N::f;
+    using N::type;
     using N::type;
-    {
-      using N::f;
-      using N::type;
-    }
   }
 }