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

From: "Martin Storsjö" <martin@martin.st>
To: ffmpeg-devel@ffmpeg.org
Subject: Re: [FFmpeg-devel] [PATCH] checkasm: Provide enough alignment in the new motion test
Date: Tue, 28 Jun 2022 18:11:03 +0300 (EEST)
Message-ID: <a198cda7-de81-8724-2f76-e45d8cae47fd@martin.st> (raw)
In-Reply-To: <20220628101836.714922-1-martin@martin.st>

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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".