[FFmpeg-devel] [PATCH] Explicit addl in snowdsp_mmx.c

Martin Storsjö martin
Mon Oct 4 15:08:36 CEST 2010


On Mon, 4 Oct 2010, Michael Niedermayer wrote:

> On Mon, Oct 04, 2010 at 03:08:17PM +0300, ?smail D?nmez wrote:
> > On Mon, Oct 4, 2010 at 3:06 PM, ?smail D?nmez <ismail at namtrac.org> wrote:
> > 
> > >
> > >
> > > On Mon, Oct 4, 2010 at 2:57 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> > >
> > >> On Mon, Oct 04, 2010 at 02:32:43PM +0300, ?smail D?nmez wrote:
> > >> > Hi;
> > >> >
> > >> > On Mon, Oct 4, 2010 at 2:27 PM, Michael Niedermayer <michaelni at gmx.at
> > >> >wrote:
> > >> >
> > >> > > q and l
> > >> > > not addq and addl
> > >> > > thats more flexible
> > >> > >
> > >> >
> > >> > How about this? Need a better name for the define though.
> > >>
> > >> OPSIZE
> > >>
> > >> and ok if tested
> > >>
> > >
> > > Attached, fate tested on x86-64.
> > >
> > >
> > Ooops this one doesn't break indentation.
> > 
> > Thanks,
> > ismail
> 
> >  libavcodec/x86/snowdsp_mmx.c |    2 +-
> >  libavutil/x86_cpu.h          |    2 ++
> >  2 files changed, 3 insertions(+), 1 deletion(-)
> > 0b9120743e83bf4d14ea1e2b9a70e1ac4437d895  snowdsp-clang5.patch
> 
> ok

Applied

// Martin



More information about the ffmpeg-devel mailing list