aaron.ballman added a comment.
In D75844#2677457 <https://reviews.llvm.org/D75844#2677457>, @brooksmoses wrote:
> FWIW, this now causes Clang to produce an error on this code, when it didn't
> before:
>
> using namespace::foo __attribute__((deprecated("message")));
>
> I discussed this with Richard Smith, who points out that GCC does not accept
> this and it's not permitted according to the C++ standard, so this should
> probably be considered an accidental fix of a longstanding bug. With that
> said, would it be useful to add this as a tested case?
I think that would be a useful test case if only to document that we explicitly
expect the situation to be a parsing error.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75844/new/
https://reviews.llvm.org/D75844
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
