Re: [FFmpeg-devel] [PATCH] huffyuvencdsp: Convert ff_diff_bytes_mmx to yasm

2015-10-20 Thread Timothy Gu
On Tue, Oct 20, 2015 at 2:18 PM James Almer wrote: > Removing this will make the INLINE_MMXEXT if statement below fail to > compile on > builds with inline asm disabled (msvc, etc). Even with dead code > elimination in > mind you'd need at least a prototype for the relevant functions. > Just move

Re: [FFmpeg-devel] [PATCH] huffyuvencdsp: Convert ff_diff_bytes_mmx to yasm

2015-10-20 Thread James Almer
On 10/19/2015 11:11 PM, Timothy Gu wrote: > Heavily based upon ff_add_bytes by Christophe Gisquet. > --- > > Taken into account James' comment, and fixed x86_32. Also saves one additional > GPR. > > --- > libavcodec/x86/Makefile| 1 + > libavcodec/x86/huffyuvencdsp.asm | 73 > +++

[FFmpeg-devel] [PATCH] huffyuvencdsp: Convert ff_diff_bytes_mmx to yasm

2015-10-19 Thread Timothy Gu
Heavily based upon ff_add_bytes by Christophe Gisquet. --- Taken into account James' comment, and fixed x86_32. Also saves one additional GPR. --- libavcodec/x86/Makefile| 1 + libavcodec/x86/huffyuvencdsp.asm | 73 ++ libavcodec/x86/huffyuvencd