arphaman added a comment.

In https://reviews.llvm.org/D28514#651675, @ahatanak wrote:

> This is not invalid:
>
>   namespace Foo {
>       class C { };
>   }
>   namespace Bar { class C { }; }
>  
>   void foo1() {
>   using Foo::C;
>   {
>   using Bar::C;
>   }
>   }
>


I see, that could happen if the previous UsingShadow is in an outer scope. Yes, 
I guess then Foo::C using decl should hide Bar::C using shadow decl. I'll post 
the patch which checks that the two decls have the same decl context (should be 
faster than checking if a using shadow decl is contained in a using decl).


Repository:
  rL LLVM

https://reviews.llvm.org/D28514



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to