| Message ID | CAEa-L+vZk01QWX1ZCLx0EpbnpfNWVWL6m2O+Rw0QB7vRXgpzqg@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [FFmpeg-devel] checkasm/ac3dsp: add float_to_fixed24 test | expand |
| Context | Check | Description |
|---|---|---|
| yinshiyou/make_loongarch64 | success | Make finished |
| yinshiyou/make_fate_loongarch64 | success | Make fate finished |
| andriy/make_x86 | success | Make finished |
| andriy/make_fate_x86 | success | Make fate finished |
Le torstaina 23. marraskuuta 2023, 9.08.16 EET flow gg a écrit :
>
You should probably add the test case to tests/fate/checkasm.mak
On 11/23/2023 4:08 AM, flow gg wrote: > +static void check_float_to_fixed24(AC3DSPContext *c) { > +#define BUF_SIZE 1024 > + LOCAL_ALIGNED_32(int32_t, v1, [BUF_SIZE]); This one is not necessary. You can reuse dst or dst2 for the bench() as it's write only. > + LOCAL_ALIGNED_32(float, v2, [BUF_SIZE]); > + > + declare_func(void, int32_t *, const float *, unsigned int); > + > + randomize_float(v2, BUF_SIZE); > + > + if (check_func(c->float_to_fixed24, "float_to_fixed24")) { > + LOCAL_ALIGNED_32(int32_t, dst, [BUF_SIZE]); > + LOCAL_ALIGNED_32(int32_t, dst2, [BUF_SIZE]); > + > + call_ref(dst, v2, BUF_SIZE); > + call_new(dst2, v2, BUF_SIZE); > + > + if (memcmp(dst, dst2, sizeof(*dst) * 10) != 0) BUF_SIZE instead of 10. > + fail(); > + > + bench_new(v1, v2, BUF_SIZE); > + } > + > + > + report("float_to_fixed24"); > +}
> You should probably add the test case to tests/fate/checkasm.mak > This one is not necessary. You can reuse dst or dst2 for the bench() as it's write only. > Changed BUF_SIZE instead of 10. Okay, changed. James Almer <jamrial@gmail.com> 于2023年11月24日周五 01:11写道： > On 11/23/2023 4:08 AM, flow gg wrote: > > +static void check_float_to_fixed24(AC3DSPContext *c) { > > +#define BUF_SIZE 1024 > > + LOCAL_ALIGNED_32(int32_t, v1, [BUF_SIZE]); > > This one is not necessary. You can reuse dst or dst2 for the bench() as > it's write only. > > > + LOCAL_ALIGNED_32(float, v2, [BUF_SIZE]); > > + > > + declare_func(void, int32_t *, const float *, unsigned int); > > + > > + randomize_float(v2, BUF_SIZE); > > + > > + if (check_func(c->float_to_fixed24, "float_to_fixed24")) { > > + LOCAL_ALIGNED_32(int32_t, dst, [BUF_SIZE]); > > + LOCAL_ALIGNED_32(int32_t, dst2, [BUF_SIZE]); > > + > > + call_ref(dst, v2, BUF_SIZE); > > + call_new(dst2, v2, BUF_SIZE); > > + > > + if (memcmp(dst, dst2, sizeof(*dst) * 10) != 0) > > BUF_SIZE instead of 10. > > > + fail(); > > + > > + bench_new(v1, v2, BUF_SIZE); > > + } > > + > > + > > + report("float_to_fixed24"); > > +} > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". >
From 02dd534bd602ba3ec79e51070934949a98f780e2 Mon Sep 17 00:00:00 2001 From: sunyuechi <sunyuechi@iscas.ac.cn> Date: Wed, 22 Nov 2023 14:57:29 +0800 Subject: [PATCH] checkasm/ac3dsp: add float_to_fixed24 test --- tests/checkasm/Makefile | 1 + tests/checkasm/ac3dsp.c | 71 +++++++++++++++++++++++++++++++++++++++ tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + 4 files changed, 76 insertions(+) create mode 100644 tests/checkasm/ac3dsp.c diff --git a/tests/checkasm/Makefile b/tests/checkasm/Makefile index 8bc241d29b..53742c93ae 100644 --- a/tests/checkasm/Makefile +++ b/tests/checkasm/Makefile @@ -1,5 +1,6 @@ # libavcodec tests # subsystems +AVCODECOBJS-$(CONFIG_AC3DSP) += ac3dsp.o AVCODECOBJS-$(CONFIG_AUDIODSP) += audiodsp.o AVCODECOBJS-$(CONFIG_BLOCKDSP) += blockdsp.o AVCODECOBJS-$(CONFIG_BSWAPDSP) += bswapdsp.o diff --git a/tests/checkasm/ac3dsp.c b/tests/checkasm/ac3dsp.c new file mode 100644 index 0000000000..2ccfa4a9d1 --- /dev/null +++ b/tests/checkasm/ac3dsp.c @@ -0,0 +1,71 @@ +/* + * Copyright (c) 2023 Institue of Software Chinese Academy of Sciences (ISCAS). + * + * This file is part of FFmpeg. + * + * FFmpeg is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * FFmpeg is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with FFmpeg; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#include <string.h> + +#include "libavutil/mem.h" +#include "libavutil/mem_internal.h" + +#include "libavcodec/ac3dsp.h" + +#include "checkasm.h" + +#define randomize_float(buf, len) \ + do { \ + int i; \ + for (i = 0; i < len; i++) { \ + float f = (float)rnd() / (UINT_MAX >> 5) - 16.0f; \ + buf[i] = f; \ + } \ + } while (0) + +static void check_float_to_fixed24(AC3DSPContext *c) { +#define BUF_SIZE 1024 + LOCAL_ALIGNED_32(int32_t, v1, [BUF_SIZE]); + LOCAL_ALIGNED_32(float, v2, [BUF_SIZE]); + + declare_func(void, int32_t *, const float *, unsigned int); + + randomize_float(v2, BUF_SIZE); + + if (check_func(c->float_to_fixed24, "float_to_fixed24")) { + LOCAL_ALIGNED_32(int32_t, dst, [BUF_SIZE]); + LOCAL_ALIGNED_32(int32_t, dst2, [BUF_SIZE]); + + call_ref(dst, v2, BUF_SIZE); + call_new(dst2, v2, BUF_SIZE); + + if (memcmp(dst, dst2, sizeof(*dst) * 10) != 0) + fail(); + + bench_new(v1, v2, BUF_SIZE); + } + + + report("float_to_fixed24"); +} + +void checkasm_check_ac3dsp(void) +{ + AC3DSPContext c; + ff_ac3dsp_init(&c); + + check_float_to_fixed24(&c); +} diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 708119e7c6..f37c7fad3a 100644 --- a/tests/checkasm/checkasm.c +++ b/tests/checkasm/checkasm.c @@ -78,6 +78,9 @@ static const struct { { "aacpsdsp", checkasm_check_aacpsdsp }, { "sbrdsp", checkasm_check_sbrdsp }, #endif + #if CONFIG_AC3DSP + { "ac3dsp", checkasm_check_ac3dsp }, + #endif #if CONFIG_ALAC_DECODER { "alacdsp", checkasm_check_alacdsp }, #endif diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h index cfea868ff1..a4238b1dfa 100644 --- a/tests/checkasm/checkasm.h +++ b/tests/checkasm/checkasm.h @@ -43,6 +43,7 @@ #include "libavutil/timer.h" void checkasm_check_aacpsdsp(void); +void checkasm_check_ac3dsp(void); void checkasm_check_afir(void); void checkasm_check_alacdsp(void); void checkasm_check_audiodsp(void); -- 2.43.0