[MPlayer-cvslog] CVS: main/liba52 liba52_amd64_changes.diff, 1.1, 1.2

[Reimar Döffinger]

Hi,
On Wed, Aug 10, 2005 at 12:56:59AM +0200, Aurelien Jacobs CVS wrote:
> CVS change done by Aurelien Jacobs CVS
> 
> Update of /cvsroot/mplayer/main/liba52
> In directory mail:/var2/tmp/cvs-serv13732/liba52
> 
> Modified Files:
> 	liba52_amd64_changes.diff 
> Log Message:
> sync liba52_amd64_changes.diff with latest fix
> 
> Index: liba52_amd64_changes.diff
> ===================================================================
> RCS file: /cvsroot/mplayer/main/liba52/liba52_amd64_changes.diff,v
> retrieving revision 1.1
> retrieving revision 1.2
> diff -u -r1.1 -r1.2
> --- liba52_amd64_changes.diff	5 Aug 2005 13:37:32 -0000	1.1
> +++ liba52_amd64_changes.diff	9 Aug 2005 22:56:57 -0000	1.2
> @@ -1182,7 +1182,7 @@
>  +		"sub $16, %%"REG_D"			\n\t"
>  +		"jnc 1b				 	\n\t"
>  +		"pop %%"REG_BP"				\n\t"//no we didnt touch ebp *g*
> -+		:: "r" (data), "r" (buf)
> ++		:: "b" (data), "c" (buf)
>  +		: "%"REG_S, "%"REG_D, "%"REG_a, "%"REG_d

Actually I think this should be necessary even withoug the AMD64 fixes
(using "r" here means it might be using ebp, which will be overwritten).
Can somebody report it upstream?

Greetings,
Reimar Döffinger