On Sat, Mar 14, 2015 at 07:58:29PM +0100, Christophe Gisquet wrote:
> Hi,
>
> 2015-03-14 19:07 GMT+01:00 Michael Niedermayer <michae...@gmx.at>:
> >> #define PR_WRAP(INSN) \
> >> static void ff_prores_idct_put_10_##INSN##_wrap(int16_t *dst){ \
> >> - DECLARE_ALIGNED(16, static int16_t, qmat)[64]; \
> >> - DECLARE_ALIGNED(16, static int16_t, tmp)[64]; \
> >> + LOCAL_ALIGNED(16, static int16_t, qmat, [64]); \
> >> + LOCAL_ALIGNED(16, static int16_t, tmp, [64]); \
> >> int i; \
> >
> > LOCAL_ALIGNED + static looks unintended
>
> Same fix then.
>
> Best regards,
> Christophe> ac3dsp_init.c | 2 +- > cavsdsp.c | 4 ++-- > dct-test.c | 4 ++-- > h264_qpel.c | 22 +++++++++++----------- > rv40dsp_init.c | 2 +- > vc1dsp_mmx.c | 2 +- > vp8dsp_init.c | 4 ++-- > 7 files changed, 20 insertions(+), 20 deletions(-) > d9bbc60366078bf5297a7c79aaf91d3ac3cc09d4 > 0002-x86-lavc-use-LOCAL_ALIGNED-instead-of-DECLARE_ALIGNE.patch > From 93a8d803f6b87e2c5bd062724630e1d67804da29 Mon Sep 17 00:00:00 2001 > From: Christophe Gisquet <christophe.gisq...@gmail.com> > Date: Sat, 14 Mar 2015 14:30:18 +0100 > Subject: [PATCH 2/3] x86: lavc: use LOCAL_ALIGNED instead of DECLARE_ALIGNED > > The later may yield incorrect code for on-stack variables. applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Breaking DRM is a little like attempting to break through a door even though the window is wide open and the only thing in the house is a bunch of things you dont want and which you would get tomorrow for free anyway
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
