[PATCH] D154503: [Sema] Fix handling of functions that hide classes

[Shafik Yaghmour via Phabricator via cfe-commits]

shafik added inline comments.


================
Comment at: clang/test/SemaCXX/using-hiding.cpp:20
+
+// Using declaration causes A::X to be hidden, so X is not ambiguous.
+namespace Test2 {
----------------
I think [namespace.udecl p10](https://eel.is/c++draft/namespace.udecl#10) 
disagrees, specifically:

```
 using A::g;                           // error: conflicts with B​::​g
```

but I may be misreading CC @Endill who has been looking at p1787r6 in details 
where a lot of this wording changed.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154503/new/

https://reviews.llvm.org/D154503

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