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

Nico Weber thakis at chromium.org
Tue Dec 2 17:26:19 PST 2014


r223186, thanks!

On Tue, Dec 2, 2014 at 5:13 PM, Richard Smith <richard at metafoo.co.uk> wrote:

> +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/0a1a65d2/attachment.html>


More information about the cfe-commits mailing list