[FFmpeg-devel] [PATCH] WMA: use type punning and unroll loops in decode_exp_vlc()

Michael Niedermayer michaelni
Tue Sep 29 11:15:44 CEST 2009


On Tue, Sep 29, 2009 at 09:10:49AM +0100, M?ns Rullg?rd wrote:
> Mans Rullgard <mans at mansr.com> writes:
> 
> > GCC does stupid things if these assignments are done using floats
> > directly, so fill the runs using integer operations instead.  Also
> > unroll the loops since the length is always a multiple of 4.
> > ---
> >  libavcodec/wmadec.c |   22 ++++++++++++++++------
> >  1 files changed, 16 insertions(+), 6 deletions(-)
> >
> > +        iv = AV_RN32(ptab + last_exp);
> 
> I don't know what I was thinking when I wrote that.  I've changed it
> to use a uint32_t pointer instead.

float->uint32_t is ok

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- 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/20090929/906df61e/attachment.pgp>



More information about the ffmpeg-devel mailing list