[FFmpeg-devel] [PATCH] Explicity use cmpl in yadif.c

Michael Niedermayer michaelni
Sun Oct 3 21:18:08 CEST 2010


On Sun, Oct 03, 2010 at 09:54:17PM +0300, ?smail D?nmez wrote:
> Hi;
> 
> clang fails to compile yadif.c,
> 
> error: ambiguous instructions require an explicit suffix (could be 'cmpb',
> ?? ? ? ? ? ? ? 'cmpw', 'cmpl', or 'cmpq')
> 
> I asked in #llvm and they said one need to let the assembler know the
> exact width of the compare,
> hence use cmpl. Attached patch does this.
> 
> Any complaints?
> 
> Regards,
> ismail

>  yadif.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> fba9f77da4fc78ae3d3d1b9cdf9faf249a63f099  yadif-clang.patch
> diff --git a/libavfilter/x86/yadif.c b/libavfilter/x86/yadif.c

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101003/3922474e/attachment.pgp>



More information about the ffmpeg-devel mailing list