[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]

Hi,

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

> 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.

Ronald