[FFmpeg-cvslog] r25196 - in trunk: configure doc/filters.texi libavfilter/Makefile libavfilter/allfilters.c libavfilter/vf_yadif.c libavfilter/x86 libavfilter/x86/yadif.c libavfilter/yadif.h

[Ronald S. Bultje]

Hi,

On Mon, Sep 27, 2010 at 8:31 PM, Alexander Strange
<astrange at ithinksw.com> wrote:
> On Sep 27, 2010, at 1:46 PM, Ronald S. Bultje wrote:
>> On Sat, Sep 25, 2010 at 12:43 PM, michael <subversion at mplayerhq.hu> wrote:
>>> Author: michael
>>> Date: Sat Sep 25 18:43:42 2010
>>> New Revision: 25196
>>>
>>> Log:
>>> yadif filter, based on stefanos port of my yadif from mplayer.
>>> Compared to stefanos, 2 frame output works with ffplay.
>>
>> http://fate.ffmpeg.org/x86_32-macosx-gcc-4.2.1/20100927154653
>>
>> Broke OSX build. Haven't had time to look at it yet.
>
>> ? ? ? ? ? ? :[prev] "r"(prev),\
>> ? ? ? ? ? ? ?[cur] ?"r"(cur),\
>> ? ? ? ? ? ? ?[next] "r"(next),\
>> ? ? ? ? ? ? ?[prefs]"r"((x86_reg)refs),\
>> ? ? ? ? ? ? ?[mrefs]"r"((x86_reg)-refs),\
>> ? ? ? ? ? ? ?[pw1] ?"m"(pw_1),\
>> ? ? ? ? ? ? ?[pb1] ?"m"(pb_1),\
>
>
> 7 registers with PIC.
> Moving pw/pb_1 to MANGLE() fixes clang -m32 -fomit-frame-pointer.
> 5 registers should work, but removing -fomit-frame-pointer too still makes it fail.

That looks right. Patch looks good to me but IANTM.

Ronald