lebedev.ri added a comment. In https://reviews.llvm.org/D51949#1248861, @JonasToth wrote:
> In https://reviews.llvm.org/D51949#1248850, @lebedev.ri wrote: > > > I have looked through tests, and it is possible that i just missed it, but > > does it test/handle the cases like: > > > > struct S { > > int X, Y, Z; // <- should be diagnosed. > > } > > > > > Unfortunatly not :( The reason is, that `int X, Y, Z;` is not a `DeclStmt` > but they are all `FieldDecl` (see https://godbolt.org/z/SXbQCa) which is why > I excluded them for now. > I did not look further into the issue, it could be easy to fix, but I don't > know. Negative tests are great to have too :) Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D51949 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
