[FFmpeg-devel] [PATCH] checkasm: Provide enough alignment in the new motion test

Martin Storsjö martin at martin.st
Tue Jun 28 18:11:03 EEST 2022


On Tue, 28 Jun 2022, Martin Storsjö wrote:

> This fixes the checkasm test in some setups on x86.
> ---
> tests/checkasm/motion.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/checkasm/motion.c b/tests/checkasm/motion.c
> index f337dd6f95..0112822174 100644
> --- a/tests/checkasm/motion.c
> +++ b/tests/checkasm/motion.c
> @@ -48,8 +48,8 @@ static void test_motion(const char *name, me_cmp_func test_func)
>     int i, x, y, d1, d2;
>     uint8_t *ptr;
>
> -    LOCAL_ALIGNED_8(uint8_t, img1, [WIDTH * HEIGHT]);
> -    LOCAL_ALIGNED_8(uint8_t, img2, [WIDTH * HEIGHT]);
> +    LOCAL_ALIGNED_16(uint8_t, img1, [WIDTH * HEIGHT]);
> +    LOCAL_ALIGNED_16(uint8_t, img2, [WIDTH * HEIGHT]);
>
>     declare_func_emms(AV_CPU_FLAG_MMX, int, struct MpegEncContext *c,
>                       uint8_t *blk1 /* align width (8 or 16) */,
> -- 
> 2.25.1

OKd on irc, and pushed.

// Martin


More information about the ffmpeg-devel mailing list