[FFmpeg-cvslog] r10234 - trunk/libavcodec/i386/snowdsp_mmx.c

Michael Niedermayer michaelni
Mon Aug 27 05:11:47 CEST 2007


Hi

On Mon, Aug 27, 2007 at 02:35:36AM +0200, Balatoni Denes wrote:
> Hi!
> 
> Luca Barbato pointed me to this part:
> 
> Sunday 26 August 2007 13:16-kor michael ezt ?rta:
> > +        "pcmpeqw    %%mm0, %%mm0                     \n\t"
> > +        "pcmpeqw    %%mm2, %%mm2                     \n\t"
> > +        "paddw      %%mm2, %%mm2                     \n\t"
> > +        "paddw      %%mm0, %%mm2                     \n\t"
> > +        "psllw        $13, %%mm2                     \n\t"
> 
> isn't the 2 paddw kind of redundant in this form? mm2 will have 
> 0x0007000700070007 no matter what.

?

mm2 is 0xA000A000A000A000

and the code should maybe be replaced by a memory read of a constant ...
i need to benchmark that ... after a few hours sleep ....

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

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- 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-cvslog/attachments/20070827/4a5f4406/attachment.pgp>



More information about the ffmpeg-cvslog mailing list