[PATCH] Implements Parts of Intrin.h

Eli Friedman eli.friedman at gmail.com
Thu Sep 26 18:41:24 PDT 2013


On Thu, Sep 26, 2013 at 6:10 PM, Reid Kleckner <rnk at google.com> wrote:

> Because it's already considered a system header, or because it shouldn't
> have warnings?
>

Because it should already be considered a system header.

-Eli


>
> The header does have warnings, because other system headers provide
> prototypes for these intrinsics, and they don't apply static, inline, or
> __forceinline / always_inline.
>
> We may have only ween warnings as a result of including this file directly.
>
>
> On Thu, Sep 26, 2013 at 6:01 PM, Eli Friedman <eli.friedman at gmail.com>wrote:
>
>>
>>   The "#pragma clang system_header" shouldn't be necessary.
>>
>> http://llvm-reviews.chandlerc.com/D1766
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130926/518976fe/attachment.html>


More information about the cfe-commits mailing list