Remove obsolete tests from acos-inputs, since slow path was removed.

Message ID 20210726115305.2360051-1-Paul.Zimmermann@inria.fr
State Superseded
Headers
Series Remove obsolete tests from acos-inputs, since slow path was removed. |

Checks

Context Check Description
dj/TryBot-apply_patch success Patch applied to master at the time it was sent
dj/TryBot-32bit fail Patch caused testsuite regressions

Commit Message

Paul Zimmermann July 26, 2021, 11:53 a.m. UTC
  ---
 benchtests/acos-inputs | 9 ---------
 1 file changed, 9 deletions(-)
  

Comments

Siddhesh Poyarekar July 26, 2021, 12:28 p.m. UTC | #1
On 7/26/21 5:23 PM, Paul Zimmermann wrote:
> ---
>   benchtests/acos-inputs | 9 ---------
>   1 file changed, 9 deletions(-)
> 
> diff --git a/benchtests/acos-inputs b/benchtests/acos-inputs
> index c90dba57ca..41e4732eb8 100644
> --- a/benchtests/acos-inputs
> +++ b/benchtests/acos-inputs
> @@ -2701,12 +2701,3 @@
>   0x1.eb1881275a5fap-1
>   0x1.e5a3c6b686c17p-1
>   -0x1.5abd34f01cc55p-25
> -# acos multiple precision fallback.
> -# Implemented in __cos32 in sysdeps/ieee754/dbl-64/sincos32.c
> -## name: slow

Please just remove the above comments and name and keep the inputs.

> -0x1.91744378d312fp-25
> -0x1.e31933dfcae5ep-6
> -0x1.560a94c69898dp-28
> -0x1.4ede34c4c5b9cp-39
> --0x1.b5caf13967673p-28
> --0x1.d242039676722p-32
>
  
Paul Zimmermann July 26, 2021, 12:45 p.m. UTC | #2
thank you Siddhesh for your review, indeed it makes sense to keep the inputs.
I've just sent a v2 (but forgot to name it v2).

Paul
  

Patch

diff --git a/benchtests/acos-inputs b/benchtests/acos-inputs
index c90dba57ca..41e4732eb8 100644
--- a/benchtests/acos-inputs
+++ b/benchtests/acos-inputs
@@ -2701,12 +2701,3 @@ 
 0x1.eb1881275a5fap-1
 0x1.e5a3c6b686c17p-1
 -0x1.5abd34f01cc55p-25
-# acos multiple precision fallback.
-# Implemented in __cos32 in sysdeps/ieee754/dbl-64/sincos32.c
-## name: slow
-0x1.91744378d312fp-25
-0x1.e31933dfcae5ep-6
-0x1.560a94c69898dp-28
-0x1.4ede34c4c5b9cp-39
--0x1.b5caf13967673p-28
--0x1.d242039676722p-32