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

Michael Niedermayer michaelni
Mon Apr 13 15:55:53 CEST 2009


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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090413/0f12d870/attachment.pgp>



More information about the ffmpeg-devel mailing list