[FFmpeg-devel] [PATCH] checkasm: sbrdsp: Fix a spurious test failure by calculating a better epsilon for sum_square

Martin Storsjö martin at martin.st
Fri Feb 7 19:59:16 EET 2020


On Wed, 5 Feb 2020, Martin Storsjö wrote:

> ---
> tests/checkasm/sbrdsp.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/tests/checkasm/sbrdsp.c b/tests/checkasm/sbrdsp.c
> index 558f452c9b..516b9f0ec6 100644
> --- a/tests/checkasm/sbrdsp.c
> +++ b/tests/checkasm/sbrdsp.c
> @@ -17,6 +17,7 @@
>  */
> 
> #include "libavcodec/sbrdsp.h"
> +#include <float.h>
> 
> #include "checkasm.h"
> 
> @@ -51,13 +52,14 @@ static void test_sum_square(void)
>     INTFLOAT res0;
>     INTFLOAT res1;
>     LOCAL_ALIGNED_16(INTFLOAT, src, [256], [2]);
> +    double t = 4 * 256;
>
>     declare_func_float(INTFLOAT, INTFLOAT (*x)[2], int n);
>
>     randomize((INTFLOAT *)src, 256 * 2);
>     res0 = call_ref(src, 256);
>     res1 = call_new(src, 256);
> -    if (!float_near_abs_eps(res0, res1, EPS))
> +    if (!float_near_abs_eps(res0, res1, t * 2 * FLT_EPSILON))
>         fail();
>     bench_new(src, 256);
> }
> -- 
> 2.17.1

Will apply soon if there's no objections

// Martin


More information about the ffmpeg-devel mailing list