95748 2012-09-04 07:50:15 -0700 TestWebKitAPI's WTF.Lrint has four failing asserts on Android 2012-09-12 03:05:55 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 peter peter d-r oldest_to_newest 711117 0 peter 2012-09-04 07:50:15 -0700 Revision 127001 introduced the WTF.Lrint tests, which seem to have a 32/64bit mismatch when run on Android. Both sizeof(long int) and sizeof(long) yield 32 and it seems to default to the NDK's lrint(). I don't have time to follow up immediately, but will eventually get around to doing it. Dominik, since these tests aren't being run on any bots yet it's not reasonable to ask you to take a look, but if you have any immediate ideas that'd be most welcome of course :-). E/chromium(13749): [ RUN ] WTF.Lrint E/chromium(13749): *** Failure in ../../Tools/TestWebKitAPI/Tests/WTF/MathExtras.cpp:44 E/chromium(13749): Value of: pow(2.0, 52) E/chromium(13749): Actual: 4.5036e+15 E/chromium(13749): Expected: lrint(pow(2.0, 52) - 0.5) E/chromium(13749): Which is: 2147483647 E/chromium(13749): *** Failure in ../../Tools/TestWebKitAPI/Tests/WTF/MathExtras.cpp:45 E/chromium(13749): Value of: pow(2.0, 52) - 2 E/chromium(13749): Actual: 4.5036e+15 E/chromium(13749): Expected: lrint(pow(2.0, 52) - 1.5) E/chromium(13749): Which is: 2147483647 E/chromium(13749): *** Failure in ../../Tools/TestWebKitAPI/Tests/WTF/MathExtras.cpp:47 E/chromium(13749): Value of: -pow(2.0, 52) E/chromium(13749): Actual: -4.5036e+15 E/chromium(13749): Expected: lrint(-pow(2.0, 52) + 0.5) E/chromium(13749): Which is: -2147483648 E/chromium(13749): *** Failure in ../../Tools/TestWebKitAPI/Tests/WTF/MathExtras.cpp:48 E/chromium(13749): Value of: -pow(2.0, 52) + 2 E/chromium(13749): Actual: -4.5036e+15 E/chromium(13749): Expected: lrint(-pow(2.0, 52) + 1.5) E/chromium(13749): Which is: -2147483648 E/chromium(13749): [ FAILED ] WTF.Lrint 718131 1 peter 2012-09-12 03:05:55 -0700 This was fixed in another patch by adding a sizeof(long int)==8 check around these asserts.