[FFmpeg-devel] [PATCH 2/4] x86: xvid_idct: port MMX IDCT to yasm

Christophe Gisquet christophe.gisquet at gmail.com
Thu Mar 12 20:13:53 CET 2015


Hi,

2015-03-11 17:06 GMT+01:00 James Almer <jamrial at gmail.com>:
> As discussed before, mmx, mmxext and sse functions that have also an sse2 version, and
> 3dnow/3dnowext functions that have also an sse version, have no reason to exist in an
> x86_64 build.

I think the reason may have been someone tinkering on some parts, and
testing them, could still see he broke the MMX versions.

On the other hand, that someone should really test on the appropriate
platform, which is easy on x86.

> They will not be used in any real world scenario.

Agreed, so I left the restriction.

Patch updated because of the dct-test stuff.

Incidentally, I found other issues with dct-test, but they are outside
the scope of this patch series.

-- 
Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-x86-xvid_idct-port-MMX-iDCT-to-yasm.patch
Type: text/x-patch
Size: 55657 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150312/6f85a616/attachment.bin>


More information about the ffmpeg-devel mailing list