https://github.com/erichkeane commented:

I believe our feedback on the last one was to emit the namespace in all 
attribute ignored diagnostics, not just if we don't know the namespace.  So 
unknown attribute `clang::whatever` as well?

Otherwise, I think this is a reasonable patch that makes this diagnostic 
somewhat more clear.

https://github.com/llvm/llvm-project/pull/138519
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to