[FFmpeg-devel] [PATCH V8 1/2] libswscale/x86/yuv2rgb: Change inline assembly into nasm code

[Fu, Ting]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> Michael Niedermayer
> Sent: Tuesday, January 21, 2020 05:04 PM
> To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH V8 1/2] libswscale/x86/yuv2rgb: Change
> inline assembly into nasm code
> 
> On Sun, Jan 19, 2020 at 11:51:03AM +0800, Ting Fu wrote:
> > The original inline assembly and nasm code have the same fps when called by
> command.
> > NASM code almost has no impact on the perfromance.
> >
> > Signed-off-by: Ting Fu <ting.fu at intel.com>
> > ---
> > V8:
> >     Remove all reindention to make review easier.
> >     Fix some improper indention.
> >     Reserve the "inline" for next patch.
> >
> >  libswscale/x86/Makefile           |   1 +
> >  libswscale/x86/swscale.c          |  16 +-
> >  libswscale/x86/yuv2rgb.c          |  26 +-
> >  libswscale/x86/yuv2rgb_template.c | 392 +++++-------------------------
> >  libswscale/x86/yuv_2_rgb.asm      | 270 ++++++++++++++++++++
> >  5 files changed, 351 insertions(+), 354 deletions(-)  create mode
> > 100644 libswscale/x86/yuv_2_rgb.asm
> 
> Seems to work, i intend to apply this in a few days
> 
> thx

Ping.

Thank you.

> 
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> 
> I do not agree with what you have to say, but I'll defend to the death your right
> to say it. -- Voltaire