[clang] [Clang] raise extension warning for unknown namespaced attributes (PR #120925)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 27 02:00:50 PST 2025


cor3ntin wrote:

Ultimately we want

1.  I know all of these attributes, don't tell me about them
2.  I know all of these attribute namespaces, don't tell me about them - which this patch is doing
3.   Have a way to disable unknown attributes warnings for all unknown namespaces - which this patch is doing by introducing a separate flag

I sort of question the last bullet point (the original issue), as it's a bit of a nuclear option. But maybe that's fine.
Then there is the question of whether 1. and 2. should have separate interfaces.

I will also point out that `-Wunknown-attribute-namespaces=foo` is a very weird way to spell "I know about `foo`, don't warn on it" - and I am concerned about discoverability.


And again, I apologize I did not see this patch sooner @a-tarasyuk.
What do people think about landing it early in the Clang 21 cycle, in case we find a better long term interface in the next few months?


https://github.com/llvm/llvm-project/pull/120925


More information about the cfe-commits mailing list