2019-01-04 20:43 GMT+01:00, Lauri Kasanen :
> The existing VSX yuv2plane1 is also ifdefed out for POWER7,
> even though it works there.
> This is for cleanliness mainly, separating the macros would
> be a bit uglier.
Please make it a bit uglier.
Sorry for missing this paragraph, Carl Eugen
_
On Sun, 6 Jan 2019 13:23:43 +0100
Carl Eugen Hoyos wrote:
> 2019-01-04 20:43 GMT+01:00, Lauri Kasanen :
> > +#ifdef __POWER8_VECTOR__
>
> If this is correct, I assume it fixes a bug in the current code
> and should be a separate patch, no?
>
> > case 16:
> > c->yuv2plane1
2019-01-04 20:43 GMT+01:00, Lauri Kasanen :
> +#ifdef __POWER8_VECTOR__
If this is correct, I assume it fixes a bug in the current code
and should be a separate patch, no?
> case 16:
> c->yuv2plane1 = isBE(dstFormat) ? yuv2plane1_16BE_vsx :
> yuv2plane1_16LE_vsx;
> +
./ffmpeg_g -f rawvideo -pix_fmt rgb24 -s hd1080 -i /dev/zero -pix_fmt
yuv420p16be \
-s 1920x1728 -f null -vframes 100 -v error -nostats -
9-14 bit funcs get about 6x speedup, 16-bit gets about 15x.
Fate passes, each format tested with an image to video conversion.
Only POWER8 includes 32-bit vec