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

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 7 23:18:01 PDT 2023


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



More information about the cfe-commits mailing list