[Patch] Test and set wchar_t related macros in -fms-extensions mode

Hans Wennborg hans at chromium.org
Thu May 2 06:13:07 PDT 2013


On Thu, May 2, 2013 at 1:52 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> The builtin and stddef.h types are compatible, right? That way, when
> including stddef.h in ms c++ mode, the typedef
>
> typedef __WCHAR_TYPE__ wchar_t;
>
> is just a nop, correct?

That typedef in stddef.h is guarded by an "#ifndef __cplusplus", so
there should be no attempt to typedef the builtin.

> LGTM if so.

Thanks! r180918.

 - Hans


> On 2 May 2013 06:30, Hans Wennborg <hans at chromium.org> wrote:
>> Hi all,
>>
>> The attached patch adds tests to check that we set the
>> _WCHAR_T_DEFINED and _NATIVE_WCHAR_T_DEFINED macros correctly in
>> -fms-extensions mode, and also teaches stddef.h to set the former when
>> typedef'ing wchar_t.
>>
>> Please take a look!
>>
>> Thanks,
>> Hans




More information about the cfe-commits mailing list