Various bug fixes.
The most amusing fix was the wierd NaN test failure. After spending way
too much time debugging, it turned out to be a missing comma in the argument
list of the test launcher.
Change-Id: I76253575d7fbe2c2c260f1839a517c1b93dc9224
diff --git a/src/compiler_test.cc b/src/compiler_test.cc
index 186e4c4..2a47936 100644
--- a/src/compiler_test.cc
+++ b/src/compiler_test.cc
@@ -120,12 +120,10 @@
38);
}
-#if 0 // Needs artFillArrayDataNoThrow() helper function
TEST_F(CompilerTest, ShiftTest1) {
CompileDex(kIntMathDex, "kIntMathDex");
AssertStaticIntMethod("IntMath", "shiftTest1", "()I", 0);
}
-#endif
TEST_F(CompilerTest, ShiftTest2) {
CompileDex(kIntMathDex, "kIntMathDex");
@@ -201,14 +199,12 @@
-5LL, -4294967287LL, 4LL, 8LL);
}
-#if 0 // Weird NaN failure. Needs investigation
TEST_F(CompilerTest, FloatCompare) {
CompileDex(kIntMathDex, "kIntMathDex");
- AssertStaticIntMethod("IntMath", "testFloatCompare", "(FFFF)I", 3333
+ AssertStaticIntMethod("IntMath", "testFloatCompare", "(FFFF)I", 3333,
-5.0f, 4.0f, 4.0f,
(1.0f/0.0f) / (1.0f/0.0f));
}
-#endif
TEST_F(CompilerTest, DoubleCompare) {
CompileDex(kIntMathDex, "kIntMathDex");