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

[Måns Rullgård]

"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.

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