On Wed, Oct 19, 2016 at 10:18:07AM +0200, Diego Biurrun wrote: > --- /dev/null > +++ b/tests/checkasm/hevc_add_res.c > @@ -0,0 +1,84 @@ > +static void check_add_res(HEVCDSPContext h, int bit_depth) > +{ > + int i; > + LOCAL_ALIGNED(32, int16_t, res0, [32 * 32]); > + LOCAL_ALIGNED(32, int16_t, res1, [32 * 32]); > + LOCAL_ALIGNED(32, uint8_t, dst0, [32 * 32 * 2]); > + LOCAL_ALIGNED(32, uint8_t, dst1, [32 * 32 * 2]); > + > + for (i = 2; i <= 5; i++) { > + int block_size = 1 << i; > + int size = block_size * block_size; > + declare_func_emms(AV_CPU_FLAG_MMX, void, uint8_t *dst, int16_t *res, > ptrdiff_t stride);
declare_func_emms should match the function declaration for the parameter names as well, i.e. given void (*add_residual[4])(uint8_t *dst, int16_t *coeffs, ptrdiff_t stride); the parameter name should be "coeffs", not "res" and the variable names updated accordingly. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel