Re: [FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-05-13 Thread Ramiro Polla
On Mon, May 13, 2024 at 6:49 PM James Almer wrote: > On 5/6/2024 2:49 PM, Rémi Denis-Courmont wrote: > > Le maanantaina 6. toukokuuta 2024, 20.18.11 EEST Ramiro Polla a écrit : > >> I'll send a similar patch to fix checkasm/idctdsp after this is merged. > > > > The idctdsp test does not actually t

Re: [FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-05-13 Thread James Almer
On 5/6/2024 2:49 PM, Rémi Denis-Courmont wrote: Le maanantaina 6. toukokuuta 2024, 20.18.11 EEST Ramiro Polla a écrit : I'll send a similar patch to fix checkasm/idctdsp after this is merged. The idctdsp test does not actually test the iDCT, but only the trivial-ish add/put helpers, so it does

Re: [FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-05-06 Thread Rémi Denis-Courmont
Le maanantaina 6. toukokuuta 2024, 20.18.11 EEST Ramiro Polla a écrit : > I'll send a similar patch to fix checkasm/idctdsp after this is merged. The idctdsp test does not actually test the iDCT, but only the trivial-ish add/put helpers, so it does not care about the context. You're welcome to fi

Re: [FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-05-06 Thread Ramiro Polla
On Thu, May 2, 2024 at 8:05 PM Rémi Denis-Courmont wrote: > Le keskiviikkona 17. huhtikuuta 2024, 21.01.37 EEST Ramiro Polla a écrit : [...] > > +static void check_fdct(void) > > +{ > > +LOCAL_ALIGNED_16(int16_t, block0, [64]); > > +LOCAL_ALIGNED_16(int16_t, block1, [64]); > > + > > +A

Re: [FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-05-02 Thread Rémi Denis-Courmont
Le keskiviikkona 17. huhtikuuta 2024, 21.01.37 EEST Ramiro Polla a écrit : > Reviewed-by: Martin Storsjö > --- > tests/checkasm/Makefile | 1 + > tests/checkasm/checkasm.c | 3 ++ > tests/checkasm/checkasm.h | 1 + > tests/checkasm/fdctdsp.c | 68 +++ > t

[FFmpeg-devel] [PATCH v3 1/2] checkasm: add test for fdct

2024-04-17 Thread Ramiro Polla
Reviewed-by: Martin Storsjö --- tests/checkasm/Makefile | 1 + tests/checkasm/checkasm.c | 3 ++ tests/checkasm/checkasm.h | 1 + tests/checkasm/fdctdsp.c | 68 +++ tests/fate/checkasm.mak | 1 + 5 files changed, 74 insertions(+) create mode 100644 te