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

Reply via email to