[patch] add support for has_feature(cxx_alignof) and has_feature(c_alignof)

Richard Smith richard at metafoo.co.uk
Tue Dec 2 17:13:07 PST 2014


+Use ``__has_feature(cxx_alignof)`` or ``__has_extension(cxx_alignof)`` to
+determine if support for alignment specifiers using ``alignof`` is enabled.

`alignof` is not an alignment-specifier. Maybe "... to determine if support
for the ``alignof`` keyword is enabled."?

Likewise for _Alignof.

Otherwise, this seems fine to me.

On Tue, Dec 2, 2014 at 4:40 PM, Nico Weber <thakis at chromium.org> wrote:

> ping!
>
> On Mon, Dec 1, 2014 at 1:07 PM, Nico Weber <thakis at chromium.org> wrote:
>
>> r142020 added support for has_feature(cxx_alignas). This patch does the
>> same for alignof.
>>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141202/3106f0e6/attachment.html>


More information about the cfe-commits mailing list