[PATCH v2 01/25] benchtests: Add acosf benchmark

DJ Delorie dj@redhat.com
Fri Dec 6 23:28:34 GMT 2024


Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:
> The input is based on acosf one (random inputs in [-1,1]).

What?  This *is* the acosf one.  And we already have acosf-inputs in
benchtests/libmvec/ but this doesn't match that.  Can we merge them
somehow?

LGTM otherwise
Reviewed-by: DJ Delorie <dj@redhat.com>

> diff --git a/benchtests/Makefile b/benchtests/Makefile
> index 2ee5516d55..924434903a 100644
> --- a/benchtests/Makefile
> +++ b/benchtests/Makefile
> @@ -25,6 +25,7 @@ include ../Makeconfig
>  
>  bench-math := \
>    acos \
> +  acosf \

Ok.

> diff --git a/benchtests/acosf-inputs b/benchtests/acosf-inputs
> new file mode 100644
> index 0000000000..bda4c44162
> --- /dev/null
> +++ b/benchtests/acosf-inputs
> @@ -0,0 +1,2710 @@
> +## args: float
> +## ret: float
> +## includes: math.h
> +# Random inputs in [-1,1]
> +## name: workload-core-math1

Ok.

> +0x1.8c9cfp-3
> +0x1.c8d8bp-1
> +-0x1.58bbe8p-2
> . . .
> +0x1.b0384cp-1
> +0x1.3e9f5p-2
> +-0x1.00d68p-2

Ok.



More information about the Libc-alpha mailing list