[MPlayer-cvslog] CVS: main/libmpcodecs pullup.c, 1.28, 1.29 vf_decimate.c, 1.4, 1.5 vf_divtc.c, 1.5, 1.6 vf_eq.c, 1.11, 1.12 vf_eq2.c, 1.11, 1.12 vf_fspp.c, 1.14, 1.15 vf_halfpack.c, 1.9, 1.10 vf_ilpack.c, 1.7, 1.8 vf_ivtc.c, 1.7, 1.8 vf_noise.c, 1.18, 1.19

[Rich Felker]

On Sat, Apr 15, 2006 at 10:46:56PM +0200, Nicolas Plourde CVS wrote:
> CVS change done by Nicolas Plourde CVS
> 
> Update of /cvsroot/mplayer/main/libmpcodecs
> In directory mail:/var2/tmp/cvs-serv17711/libmpcodecs
> 
> Modified Files:
> 	pullup.c vf_decimate.c vf_divtc.c vf_eq.c vf_eq2.c vf_fspp.c 
> 	vf_halfpack.c vf_ilpack.c vf_ivtc.c vf_noise.c 
> Log Message:
> add support for intel mac. mp3lib is not fixed yet.
> 
> Index: pullup.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/libmpcodecs/pullup.c,v
> retrieving revision 1.28
> retrieving revision 1.29
> diff -u -r1.28 -r1.29
> --- pullup.c	9 Feb 2006 14:07:55 -0000	1.28
> +++ pullup.c	15 Apr 2006 20:46:54 -0000	1.29
> @@ -5,6 +5,7 @@
>  #include <string.h>
>  #include "pullup.h"
>  #include "config.h"
> +#include "asmalign.h"
>  
>  
>  
> @@ -18,7 +19,7 @@
>  		"pxor %%mm4, %%mm4 \n\t"
>  		"pxor %%mm7, %%mm7 \n\t"
>  		
> -		".balign 16 \n\t"
> +		ASMALIGN16
>  		"1: \n\t"
>  		
>  		"movq (%%esi), %%mm0 \n\t"
> @@ -67,7 +68,7 @@
>  		"pxor %%mm7, %%mm7 \n\t"
>  		"subl %%eax, %%edi \n\t"
>  		
> -		".balign 16 \n\t"
> +		ASMALIGN16
>  		"2: \n\t"
>  
>  		"movq (%%esi), %%mm0 \n\t"
> @@ -156,7 +157,7 @@
>  		"pxor %%mm4, %%mm4 \n\t"
>  		"pxor %%mm7, %%mm7 \n\t"
>  		
> -		".balign 16 \n\t"
> +		ASMALIGN16
>  		"1: \n\t"
>  		
>  		"movq (%%esi), %%mm0 \n\t"

External dependencies on MPlayer files are NOT ALLOWED in pullup.c.
It's to be treated as imported code. IMO just remove these align
directives. They were not needed afaik.. It's probably worth
benchmarking first.

Rich