[FFmpeg-devel] [PATCH] Fix compilation on clang 2.8, which doesn't support 3dnow inline asm.

[Måns Rullgård]

M?ns Rullg?rd <mans at mansr.com> writes:

> "Ronald S. Bultje" <rsbultje at gmail.com> writes:
>
>> Hi,
>>
>> 2011/2/11 M?ns Rullg?rd <mans at mansr.com>:
>>> "Ronald S. Bultje" <rsbultje at gmail.com> writes:
>>>
>>>> ---
>>>> ?configure ? ? ? ? ? ? ? ? ? ? ? | ? ?8 +++++---
>>>> ?libavcodec/x86/cavsdsp_mmx.c ? ?| ? ?4 ++++
>>>> ?libavcodec/x86/dsputil_mmx.c ? ?| ? 39 ++++++++++++++++++++++++++++++++-------
>>>> ?libavcodec/x86/dsputilenc_mmx.c | ? ?4 ++++
>>>> ?libavcodec/x86/fmtconvert_mmx.c | ? 10 ++++++++++
>>>> ?libavcodec/x86/h264_qpel_mmx.c ?| ? ?2 ++
>>>> ?libswscale/rgb2rgb.c ? ? ? ? ? ?| ? ?4 ++++
>>>> ?7 files changed, 61 insertions(+), 10 deletions(-)
>>>>
>>>> diff --git a/configure b/configure
>>>> index a3ca15d..48e7d60 100755
>>>> --- a/configure
>>>> +++ b/configure
>>>> @@ -2649,9 +2649,11 @@ int main(void) {
>>>> ?}
>>>> ?EOF
>>>>
>>>> - ? ?# check whether binutils is new enough to compile SSSE3/MMX2
>>>> - ? ?enabled ssse3 && check_asm ssse3 '"pabsw %xmm0, %xmm0"'
>>>> - ? ?enabled mmx2 ?&& check_asm mmx2 ?'"pmaxub %mm0, %mm1"'
>>>> + ? ?# check whether binutils is new enough to compile SSSE3/MMX2/3dnow
>>>> + ? ?enabled ssse3 ? ?&& check_asm ssse3 ? ?'"pabsw ?%xmm0, %xmm0"'
>>>> + ? ?enabled mmx2 ? ? && check_asm mmx2 ? ? '"pmaxub ?%mm0, %mm1"'
>>>> + ? ?enabled amd3dnow && check_asm amd3dnow '"pavgusb %mm0, %mm1"'
>>>> + ? ?enabled amd3dnowext && enabled amd3dnow || disable amd3dnowext
>>>
>>> amd3dnowext already has a _deps on amd3dnow so that last line isn't needed.
>>
>> It remained enabled without it for some obscure reason.
>
> Then that reason should be found and corrected.

configure --disable-amd3dnow correctly disables amd3dnowext here.
Does that work on your machine?

-- 
M?ns Rullg?rd
mans at mansr.com