r190979 - Avoid including <stdlib.h> in the intrin.h test

Reid Kleckner rnk at google.com
Wed Sep 18 18:12:07 PDT 2013


On Wed, Sep 18, 2013 at 6:05 PM, Eli Friedman <eli.friedman at gmail.com>wrote:

> On Wed, Sep 18, 2013 at 5:32 PM, Reid Kleckner <reid at kleckner.net> wrote:
>
>> Author: rnk
>> Date: Wed Sep 18 19:32:11 2013
>> New Revision: 190979
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=190979&view=rev
>> Log:
>> Avoid including <stdlib.h> in the intrin.h test
>>
>> Modified:
>>     cfe/trunk/test/Headers/ms-intrin.cpp
>>
>> Modified: cfe/trunk/test/Headers/ms-intrin.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Headers/ms-intrin.cpp?rev=190979&r1=190978&r2=190979&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Headers/ms-intrin.cpp (original)
>> +++ cfe/trunk/test/Headers/ms-intrin.cpp Wed Sep 18 19:32:11 2013
>> @@ -1,5 +1,10 @@
>>  // RUN: %clang -target i386-pc-win32 -fms-extensions -fsyntax-only %s
>>
>> +// Get size_t, but avoid including mm_malloc.h which includes stdlib.h
>> which may
>> +// not exist.
>> +#include <stdint.h>
>> +#undef __STDC_HOSTED__
>> +
>>  #include <Intrin.h>
>>
>>
> It's more conventional to use -ffreestanding rather than write "#undef
> __STDC_HOSTED__"...
>

That doesn't work because we need size_t from stdint.h.  ;_;  I recall this
is why Eric gave up testing it in the first place.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130918/a55a02b0/attachment.html>


More information about the cfe-commits mailing list