Endill added a comment.
Thank you for taking a look at this.
================
Comment at: clang/test/CXX/drs/dr18xx.cpp:172-227
+namespace dr1894 { // dr1894: 3.8
+ // NB: reusing dr407 test
+ struct S;
+ typedef struct S S;
+ void f() {
+ struct S *p;
+ {
----------------
cor3ntin wrote:
> I think i would prefer reusing verbatim the example in 1894 - lines 207 and
> following basically - as the issue clearly state that what comes before is
> resolved by 407 so not really relevant here
Does the same apply to 2199? It also contains examples resolved by 407.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148136/new/
https://reviews.llvm.org/D148136
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
