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

Christophe Gisquet christophe.gisquet at gmail.com
Sat Jun 14 20:22:15 CEST 2014


Hi,

2014-06-14 20:06 GMT+02:00 Michael Niedermayer <michaelni at gmx.at>:
> 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

I was cargo-culting as I don't know altivec, and actually, block is
already vec_s16, so I had the offsets wrong. I only commented in case
compilation failed.

Your solution works, so unless Carl wants to take the time to verify
if the 2 syntaxes generate different instructions with objdump, yours
is fine and should be applied.

And if it happens not to be, the vec_st stuff will have to be checked.


More information about the ffmpeg-devel mailing list