[FFmpeg-devel] [PATCH 10/11] avcodec/x86: add an 8-bit simple IDCT function based on the x86-64 high depth functions

Ronald S. Bultje rsbultje at gmail.com
Sun Jun 25 01:33:55 EEST 2017


On Sat, Jun 24, 2017 at 6:30 PM, Ronald S. Bultje <rsbultje at gmail.com>

> If the second, then simple is a superset of simplemmx. If the first, then
> we should remove simplemmx from the list of "supported" idcts for the
> sse2/avx functions. I have no preference (I assumed it meant the first)

Just to explain this a little more: I assumed the first, because the C
function is assigned if simplemmx is set but cpuflags is set to 0 (or we're
not on x86). However, that could also be considered a fallback so I
understand both points of view.


More information about the ffmpeg-devel mailing list