[Ffmpeg-devel] [PATCH] Faster rgb32tobgr32

Ivo ivop
Tue Apr 17 22:42:52 CEST 2007


On Tuesday 17 April 2007 21:39, Michael Niedermayer wrote:
> On Tue, Apr 17, 2007 at 08:24:43PM +0200, Ivo wrote:
> > On Tuesday 17 April 2007 16:51, Trent Piepho wrote:
> > > Don't suppose you know the variance of the ten runs?
> >
> > Old C code,    Avg: 68521191  Var: 2213928285808
> > New C code,    Avg: 67644935  Var: 970441093200
> > Old MMX2 code, Avg: 66875441  Var: 10001941746808
> > New MMX2 code, Avg: 65855368  Var: 650060307672
> >
> > > +               : "+r"(idx)
> > > +               : "r" (s), "r" (d), "m" (mask32b), "m" (mask32r), "m"
> > > (mmx_one)
> > >
> > >                 : "memory");
> > >
> > > It should be "+&r"(idx).  Realistically, idx won't be able to overlap
> > > any of those parameters, but someone could always add a new input
> > > that could overlap.
> >
> > Fixed locally.
>
> patch ok

Applied.

--Ivo




More information about the ffmpeg-devel mailing list