[FFmpeg-devel] [PATCH] skip no-op loop iterations in ff_flac_compute_autocorr

Diego Biurrun diego
Mon Apr 13 16:03:56 CEST 2009


On Mon, Apr 13, 2009 at 03:55:53PM +0200, Michael Niedermayer wrote:
> On Mon, Apr 13, 2009 at 09:53:52AM -0400, Bobby Bingham wrote:
> > On Thu, 09 Apr 2009 18:02:12 -0400
> > Justin Ruggles <justin.ruggles at gmail.com> wrote:
> > 
> > > Bobby Bingham wrote:
> > > > On Wed, 08 Apr 2009 17:42:55 -0400
> > > > Justin Ruggles <justin.ruggles at gmail.com> wrote:
> > > > 
> > > >> Bobby Bingham wrote:
> > > >>> Attached patch skips a bunch of iterations which are no-ops
> > > >>> because data1[-lag...-1] are all zero.
> > > >> looks ok. do the ASM versions do the same?
> > > > 
> > > > Good question.  I'll take a look at deciphering them.  In the
> > > > meantime, here's an updated patch which applies the same change to
> > > > the other loop in the same function as well.
> > > 
> > > looks good.
> > 
> > Thanks.  Can somebody apply this?  AFAIK I only have commit access to
> > the SoC repository.
> 
> diego, please give bobby svn write access to ffmpeg-svn if he does not
> have

Done.

Diego



More information about the ffmpeg-devel mailing list