On Thu, Nov 10, 2016 at 02:59:15PM +0100, Diego Biurrun wrote: > On Thu, Nov 10, 2016 at 02:41:05PM +0100, Alexandra Hájková wrote: > > --- /dev/null > > +++ b/libavcodec/ppc/hevcdsp.c > > @@ -0,0 +1,107 @@ > > + if (!PPC_ALTIVEC(av_get_cpu_flags())) > > + return; > > + > > + if (bit_depth == 8) > > + c->idct[0] = ff_hevc_idct_4x4_8; > > + if (bit_depth == 10) > > + c->idct[0] = ff_hevc_idct_4x4_10; > > These functions use AltiVec, so they should have a "_altivec" suffix. > We always add a suffix indicating the SIMD type to SIMD-optimized > functions.
The name without suffix is reserved for the C functions, which not always have a "_c" suffix themselves. Diego _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel