[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