<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 17 May 2017 at 16:24, Marc Espie via cfe-dev <span dir="ltr"><<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, May 17, 2017 at 03:55:06PM -0700, Reid Kleckner wrote:<br>
<span class="">><br>
>    On Wed, May 17, 2017 at 3:22 PM, Marc Espie <[1]<a href="mailto:espie@nerim.net">espie@nerim.net</a>> wrote:<br>
><br>
>      On Wed, May 17, 2017 at 01:01:07PM -0700, Reid Kleckner wrote:<br>
>      ><br>
</span>>      >  Â  I'd rather not add new pre-defined macros if we can avoid it.<br>
>      There are<br>
>      >  Â  already too many. You can probably detect this situation<br>
>      with:<br>
>      >  Â  #if defined(_MSC_EXTENSIONS) && !defined(_WCHAR_T_DEFINED)<br>
<span class="">>      ><br>
>      Was this added recently ?<br>
>      There is no _MSC_EXTENSIONS in the clang I'm using...<br>
><br>
>    Looks like we only define it for Windows targets.<br>
>    How about __is_identifier?<br>
>    $ cat t.cpp<br>
>    #if !__is_identifier(__wchar_t)<br>
>    #error "have __wchar_t"<br>
>    #else<br>
>    #error "no __wchar_t"<br>
>    #endif<br>
<br>
</span>Nope, that will trigger errors on non clang compilers.<br></blockquote><div><br></div><div>You can wrap it in #ifdef __clang__ to fix that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Why do you fight so hard against a simple solution ?<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-dev</a><br>
</div></div></blockquote></div><br></div></div>