aaron.ballman added a comment.
In D126061#3605276 <https://reviews.llvm.org/D126061#3605276>, @miyuki wrote:
> Hi Martin,
>
> I think this patch caused a regression in diagnostics. Clang used to warn
> about the following code:
>
> struct X {
> [[deprecated]] struct Y {};
> };
>
>
>
> $ clang -std=c++17 -Wall test.cpp
> <source>:2:7: warning: attribute 'deprecated' is ignored, place it after
> "struct" to apply attribute to type declaration [-Wignored-attributes]
> [[deprecated]] struct Y {};
> ^
>
> And now it doesn't. Could you please have a look into it?
Good catch, that does appear to be a regression:
http://eel.is/c++draft/class#nt:member-declaration
http://eel.is/c++draft/class#mem.general-14
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126061/new/
https://reviews.llvm.org/D126061
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
