[PATCH] D138822: [clang] Add test for CWG36

Vlad Serebrennikov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 29 11:22:28 PST 2022


Endill added inline comments.


================
Comment at: clang/test/CXX/drs/dr0xx.cpp:489
+
+    using B::i; // expected-error {{redeclaration of using declaration}}
+    using C::i; // expected-error {{redeclaration of using declaration}}
----------------
erichkeane wrote:
> As a nit, I prefer the 'notes' to live next to the error, and use a bookmark line-marker here.  My issue is basically how we have no way of knowing (particularly in template code...) what this diagnoses.
> 
> I would also think a dependent example of this diagnostic would be useful.
>I would also think a dependent example of this diagnostic would be useful.
Do you mean something of this sort: `using D<int>::i`?


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

https://reviews.llvm.org/D138822



More information about the cfe-commits mailing list