[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