[FFmpeg-cvslog] Fix linking without yasm.

Clément Bœsch ubitux at gmail.com
Sat Mar 24 23:41:26 CET 2012


On Sat, Mar 24, 2012 at 02:55:01PM +0100, Carl Eugen Hoyos wrote:
> ffmpeg | branch: master | Carl Eugen Hoyos <cehoyos at ag.or.at> | Sat Mar 24 14:54:06 2012 +0100| [5cddfc58d882192d98da1520fdafc4869a3e8099] | committer: Carl Eugen Hoyos
> 
> Fix linking without yasm.
> 
> > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=5cddfc58d882192d98da1520fdafc4869a3e8099
> ---
> 
>  libavcodec/x86/vc1dsp_mmx.c |   10 ++++++----
>  1 files changed, 6 insertions(+), 4 deletions(-)
> 
> diff --git a/libavcodec/x86/vc1dsp_mmx.c b/libavcodec/x86/vc1dsp_mmx.c
> index 9c59a21..32891a0 100644
> --- a/libavcodec/x86/vc1dsp_mmx.c
> +++ b/libavcodec/x86/vc1dsp_mmx.c
> @@ -752,7 +752,8 @@ void ff_vc1dsp_init_mmx(VC1DSPContext *dsp)
>          dsp->put_vc1_mspel_pixels_tab[11] = put_vc1_mspel_mc32_mmx;
>          dsp->put_vc1_mspel_pixels_tab[15] = put_vc1_mspel_mc33_mmx;
>  
> -        dsp->put_no_rnd_vc1_chroma_pixels_tab[0]= ff_put_vc1_chroma_mc8_mmx_nornd;
> +        if (HAVE_YASM)
> +            dsp->put_no_rnd_vc1_chroma_pixels_tab[0]= ff_put_vc1_chroma_mc8_mmx_nornd;

I may miss something obvious but isn't this abusing the fact the compiler
will optimize out this branch? Aren't we supposed to use the cpp here?

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-cvslog/attachments/20120324/45853278/attachment.asc>


More information about the ffmpeg-cvslog mailing list