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

[Måns Rullgård]

"Rémi Denis-Courmont" <remi at remlab.net> writes:

>    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.

You can't have an explicit register both as an operand and in the
clobber list.

-- 
Måns Rullgård
mans at mansr.com