On Jul 9, 2011 9:46 AM, "Dylan Smith" <dylan.ah.smith@gmail.com> wrote:
>
> On Sat, Jul 9, 2011 at 10:16 AM, Misha Koshelev <misha680@gmail.com> wrote:
> > Does the attached patch work on MS Visual Studio 2010? This passes all tests.
>
> Ya, it should work.  I was only building test applications using the
> cl from the platform SDK in wine; I haven't been using it for make
> crosstest.
>
> > Your version does not seem to convert properly to +NAN on some
> > platforms (e.g., Linux) and results in some test failures.
>
> That doesn't make sense, because I am primarily using gcc in Ubuntu.
> Did you try it with the parentheses, because unary minus has higher
> precedence than division:
>
> -0.0f/zero => -nan (0xffc00000)
> -(0.0f/zero) => nan (0x7fc00000)

Hi Dylan:

I am sorry I miswrote about Ubuntu. However, there were some test failures that occurred on Winetestbot when I tried to use -(0.0f/zero) for nan.

Thank you
Misha