Hi, On Mon, Nov 14, 2016 at 4:46 PM, Martin Storsjö <mar...@martin.st> wrote:
> The dc-only mode is already checked to work correctly above, but this > allows benchmarking this mode for performance tuning, and allows making > sure that it actually is correctly hooked up. > --- > tests/checkasm/vp9dsp.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/tests/checkasm/vp9dsp.c b/tests/checkasm/vp9dsp.c > index 690e0cf..b9d1c73 100644 > --- a/tests/checkasm/vp9dsp.c > +++ b/tests/checkasm/vp9dsp.c > @@ -297,6 +297,12 @@ static void check_itxfm(void) > } > bench_new(dst, sz * SIZEOF_PIXEL, coef, sz * sz); > } > + if (txtp == 0 && tx != 4) { > + if (check_func(dsp.itxfm_add[tx][txtp], > "vp9_inv_%s_%dx%d_dc_add", > + txtp_types[txtp], sz, sz)) { > + bench_new(dst, sz * SIZEOF_PIXEL, coef, 1); > + } > + } > } > } > report("itxfm"); > -- > 2.7.4 I had a different local modification that allows tuning all the relevant sub-IDCTs, basically re-arranging the loops so check_func is inside the sub-IDCT loop and we bench each sub-IDCT separately. That's more generic and probably more useful. Ronald _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel