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

[Aurelien Jacobs]

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.

Aurel