[clang] [Clang] raise extension warning for unknown namespaced attributes (PR #120925)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 24 07:13:24 PST 2025
AaronBallman wrote:
> > > > That's how this already works in this patch:
> > > > AH! I missed that this was doing that. Awesome! Though I think the 'no' flag is the one that makes sense in a "turn it on for all BUT these". The flag as typed there isn't particularly useful (that is, ONLY warn me for the ones that I know about).
> > >
> > >
> > > > I think what Corentin is asking for is a similar (follow-up?) feature that allows you to say "warn me about unknown attributes, but not of unknown attributes." that can take attributes with or without a namespace.
> > >
> > >
> > > I'm not hearing a difference?
> >
> >
> > Wow. My fingers are obviously not listening to my brain very well. :-D
> > I mean "warn me about unknown attribute, but not these particular unknown attributes." e.g., (fake warning group name)
> > ```
> > // -Wwarn-about-unknown-attributes-except-these=foo,bar::baz
> > [[foo, bar::baz, quux]] int x; // warning: unknown attribute 'quux'
> > ```
>
> Ah, hrm. I see much less value in that. How likely is it that you have an attribute in a 'known' namespace that is 'unknown' but valid?
For example, we know about the `gnu` namespace, but there are a ton of attributes GCC supports but we do not.
https://github.com/llvm/llvm-project/pull/120925
More information about the cfe-commits
mailing list