On Jul 9, 2011 9:46 AM, "Dylan Smith" <dylan.ah.smith(a)gmail.com> wrote:
On Sat, Jul 9, 2011 at 10:16 AM, Misha Koshelev <misha680(a)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