ARC: Fix max ULP for cosine test

Message ID 1471945278-18247-1-git-send-email-Eugeniy.Paltsev@synopsys.com
State New
Headers show

Commit Message

Eugeniy Paltsev Aug. 23, 2016, 9:41 a.m.
Max ULP for "cos (M_PI_6l * 2.0) == 0.5" in "test-double" sub-test
is set to "0" for ARC. It causes "test-double" test failure.

I realized that this sub-test on ARM gets completely same results.
But it doesn't fails on ARM because max ULP is set to "1" for ARM.

I am wondering, if it was done for any special reason?

Otherwise, set max ULP for "cos (M_PI_6l * 2.0) == 0.5" test to
"1" instead of "0" for ARC.

Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>
---
 test/math/libm-test-ulps-arc | 1 +
 1 file changed, 1 insertion(+)

Comments

Waldemar Brodkorb Sept. 10, 2016, 12:05 p.m. | #1
Hi Eugeniy,
Eugeniy Paltsev wrote,

> Max ULP for "cos (M_PI_6l * 2.0) == 0.5" in "test-double" sub-test
> is set to "0" for ARC. It causes "test-double" test failure.
> 
> I realized that this sub-test on ARM gets completely same results.
> But it doesn't fails on ARM because max ULP is set to "1" for ARM.
> 
> I am wondering, if it was done for any special reason?
> 
> Otherwise, set max ULP for "cos (M_PI_6l * 2.0) == 0.5" test to
> "1" instead of "0" for ARC.

Thanks applied and pushed. Normally I autogenerate the ULPS files
with tst-float -u .. and then merging with the perl script.

best regards
 Waldemar

Patch

diff --git a/test/math/libm-test-ulps-arc b/test/math/libm-test-ulps-arc
index 7139447..46b2ac5 100644
--- a/test/math/libm-test-ulps-arc
+++ b/test/math/libm-test-ulps-arc
@@ -2,6 +2,7 @@ 
 
 # cos
 Test "cos (M_PI_6l * 2.0) == 0.5":
+double: 1
 idouble: 1
 Test "cos (M_PI_6l * 4.0) == -0.5":
 double: 2