r199053 - Clarify warn_cxx98_compat_attribute diagnostic

Chandler Carruth chandlerc at google.com
Mon Jan 13 19:21:09 PST 2014


On Mon, Jan 13, 2014 at 6:37 PM, Aaron Ballman <aaron at aaronballman.com>wrote:

> On Mon, Jan 13, 2014 at 9:24 PM, Richard Smith <richard at metafoo.co.uk>
> wrote:
> > On Sun, Jan 12, 2014 at 11:14 AM, Aaron Ballman <aaron at aaronballman.com>
> > wrote:
> >>
> >> After doing a bit more research and discussion off-list, I think
> >> "generalized attribute" is acceptable.  So patch LGTM as-is.
> >
> >
> > Really? I wouldn't expect someone seeing this diagnostic to understand
> that
> > "generalized attribute" means C++11 attributes (it's a really weird term,
> > since they're not a generalization of anything). This isn't an official
> name
> > for them, and doesn't distinguish them from the other attribute syntaxes
> we
> > support. Given that this is a diagnostic about compatibility with C++98,
> > "C++11 attributes" seems like the clearest way of expressing this.
>
> As Alp had pointed out, we document the name as "generalized
> attribute" in our feature support documentation,


Also, mostly as a meta-point, I'd like to not expect our feature support
documentation to be read by very many users of Clang, relatively speaking.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140113/50daa2dd/attachment.html>


More information about the cfe-commits mailing list