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

Chandler Carruth chandlerc at google.com
Wed Sep 18 18:16:43 PDT 2013


On Wed, Sep 18, 2013 at 6:12 PM, Reid Kleckner <rnk at google.com> wrote:

> 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.
>

Just define size_t yourself using
decltype.cfe-commits<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/20130918/324e7ce4/attachment.html>


More information about the cfe-commits mailing list