<br><br><div class="gmail_quote">On Mon, May 21, 2012 at 6:03 PM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Timur,<br>
<div class="im"><br>
> -#ifdef _WIN32<br>
> +#if defined(_WIN32) && !defined(__clang__)<br>
>  #include <intrin.h><br>
>  #endif<br>
</div>I think it should check for _MSC_VER here instead. And in many other<br>
places as well.<br>
<div class="im"><br>
>  #if defined(_WIN32)<br>
> +# if defined(__clang__)<br>
> +typedef int              intptr_t;<br>
> +typedef unsigned int     uintptr_t;<br>
> +# endif<br>
</div>These should go from stddef.h. What's the problem with it?<br></blockquote><div><br></div><div>It doesn't define what's needed. On windows these folks live somewhere else. </div><div>Adding other system headers brings in too much stuff which makes further porting efforts more complicated. </div>
<div>We've spend quite a bit of time geting rid of things like stdlib.h</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +#if !defined(_WIN32) || defined(__clang__)<br>
So, on Win64 you will provide all this stuff regardless of compiler?<br>
This looks plain wrong.<br>
<br>
Same problems in other places.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
With best regards, Anton Korobeynikov<br>
Faculty of Mathematics and Mechanics, Saint Petersburg State University<br>
</font></span></blockquote></div><br>