Message ID | 20220628101836.714922-1-martin@martin.st |
---|---|
State | Accepted |
Commit | 900424cda97082a0f0a4ed9ae6b081d2cef557d7 |
Headers | show |
Series | [FFmpeg-devel] checkasm: Provide enough alignment in the new motion test | expand |
Context | Check | Description |
---|---|---|
yinshiyou/make_loongarch64 | success | Make finished |
yinshiyou/make_fate_loongarch64 | success | Make fate finished |
andriy/make_x86 | success | Make finished |
andriy/make_fate_x86 | success | Make fate finished |
andriy/make_armv7_RPi4 | success | Make finished |
andriy/make_fate_armv7_RPi4 | success | Make fate finished |
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
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) */,