[vlc-devel] [PATCH] Add "ebx" to a clobber list

[Rémi Denis-Courmont]

   Hello,

On Sunday 23 October 2011, Rafaël Carré wrote:
> Le Sun, 23 Oct 2011 19:29:47 +0900,
> 
> KO Myung-Hun <komh78 at gmail.com> a écrit :
> > Hi/2.
> > 
> > This patch fixes the problem that "ebx" is corrupted on the some
> > compiler such as gcc 4.4.5.
> 
> Doesn't push/pop do this already?

AFAIK, removing the push/pop pair _and_ marking EBX clobbered is the cleanest 
correct code. Essentially, it tells the compilter to do push/pop, *if* it 
needs the EBX value.

That said, I believe (not sure) that listing a register in the output list 
implicitly marks it clobbered automatically.

> If the compiler is buggy can you paste corresponding assembler output?
> (replace -c by -S in compiler command)

-- 
Rémi Denis-Courmont
http://www.remlab.info