Re: [libav-devel] [PATCH] x86/dsputil: put inline asm under HAVE_INLINE_ASM.

2012-07-24 Thread Ronald S. Bultje
Hi, On Sun, Jul 22, 2012 at 3:27 PM, Derek Buitenhuis wrote: > On 22/07/2012 6:14 PM, Ronald S. Bultje wrote: >> From: "Ronald S. Bultje" >> >> This allows compiling with compilers that don't support gcc-style >> inline assembly. >> --- > > I think this looks OK, assuming: > > 1) You've tested e

Re: [libav-devel] [PATCH] x86/dsputil: put inline asm under HAVE_INLINE_ASM.

2012-07-22 Thread Derek Buitenhuis
On 22/07/2012 6:14 PM, Ronald S. Bultje wrote: > From: "Ronald S. Bultje" > > This allows compiling with compilers that don't support gcc-style > inline assembly. > --- I think this looks OK, assuming: 1) You've tested every yasm/inline asm enable/disable combination 2) Everybody else is OK wit

[libav-devel] [PATCH] x86/dsputil: put inline asm under HAVE_INLINE_ASM.

2012-07-22 Thread Ronald S. Bultje
From: "Ronald S. Bultje" This allows compiling with compilers that don't support gcc-style inline assembly. --- libavcodec/dct-test.c|2 +- libavcodec/x86/dsputil_mmx.c | 69 -- libavcodec/x86/h264_qpel_mmx.c |4 ++- libavcodec/x86/

[libav-devel] [PATCH] x86/dsputil: put inline asm under HAVE_INLINE_ASM.

2012-07-22 Thread Ronald S. Bultje
From: "Ronald S. Bultje" This allows compiling with compilers that don't support gcc-style inline assembly. --- libavcodec/dct-test.c|2 +- libavcodec/x86/dsputil_mmx.c | 69 -- libavcodec/x86/h264_qpel_mmx.c |4 ++- libavcodec/x86/