[FFmpeg-cvslog] r16488 - in trunk: configure libavutil/mem.c

[ramiro]

Aurelien Jacobs wrote:
> ramiro wrote:
> 
>> Michael Niedermayer wrote:
>>> On Thu, Jan 08, 2009 at 09:34:06PM +0100, Vitor Sessak wrote:
>>>> ramiro wrote:
>>>>> Author: ramiro
>>>>> Date: Thu Jan  8 00:36:34 2009
>>>>> New Revision: 16488
>>>>>
>>>>> Log:
>>>>> Use posix_memalign() if available.
>>>> By looking at FATE, it looks like this change broke H264 decoding on gcc 
>>>> 2.95.3. See for example 
>>>> http://fate.multimedia.cx/index.php?test_result=6753660 .
>>> as far as i understand it fate didnt build 3 revissions and this one
>>> does not look like the most likely cause, 16490 looks more likely
>> I couldn't get a hold of gcc 2.95 to test, but this patch should fix it.
>>
>> [...]
>>
>> Index: configure
>> ===================================================================
>> --- configure	(revision 16490)
>> +++ configure	(working copy)
>> @@ -1740,8 +1740,8 @@
>>      # check wether EBX is available on x86
>>      check_asm ebx_available '"":::"%ebx"'
>>  
>> -    # check whether binutils is new enough to compile SSSE3/MMX2
>> -    enabled ssse3 && check_asm ssse3 '"pabsw %xmm0, %xmm0"'
>> +    # check whether binutils is new enough to compile SIMD
>> +    enabled sse   && check_asm sse   '"xorps %xmm0, %xmm0"'
>>      enabled mmx2  && check_asm mmx2  '"movss %xmm0, %xmm0"'
> 
> Your new sse test might be OK, but removing the ssse3 test is
> certainly not.

Too much cutting and not enough pasting. New patch attached.

Ramiro Polla

-------------- next part --------------
A non-text attachment was scrubbed...
Name: disable_sse_3.diff
Type: text/x-patch
Size: 567 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20090108/c0dd4f8f/attachment.bin>