[FFmpeg-devel] [PATCH] avcodec/ppc/idct_altivec: add plain idct()

Michael Niedermayer michaelni at gmx.at
Sat Jun 14 20:06:24 CEST 2014


On Sat, Jun 14, 2014 at 07:32:08PM +0200, Christophe Gisquet wrote:
> Hi,
> 
> 2014-06-14 19:26 GMT+02:00 Michael Niedermayer <michaelni at gmx.at>:
> > +    block[0] = vx0;
> 
> Cargo-culting my reply from other pieces of code (eg vc1 code), maybe
> this would rather be:
> vec_st(vx0,  0, block);
> vec_st(vx1, 16, block);
> etc

i dont really know altivec / the advantages of direct assignment vs
using vec_st*() and i dont have hw to test either
so if my patch is silly in some way, please submit a correct one
and ill apply it

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

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140614/2596d970/attachment.asc>


More information about the ffmpeg-devel mailing list