Hi,

On Mon, Jun 20, 2011 at 6:37 AM, Kostya <[email protected]> wrote:
> On Mon, Jun 20, 2011 at 11:22:09AM +0100, Måns Rullgård wrote:
>> Kostya <[email protected]> writes:
>>
>> > On Mon, Jun 20, 2011 at 10:55:17AM +0100, Mans Rullgard wrote:
>> >> Nothing requires this value in %eax.
>> >
>> > IIUC, return value is put into eax, thus making asm output result directly
>> > into that register, thus preventing GCC from pessimising the code.
>>
>> The function is often inlined where there is no return value as such.
>> In the non-inline case, gcc does use eax for that value on its own.
>
> Not that I care much, but I'd like to hear from x86 experts, otherwise it
> looks fine with me.

The function seems marked with av_always_inline, so there is no return value.

Patch OK.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to