[FFmpeg-devel] [FFmpeg-devel-irc] IRC log for 2010-09-08

[Reimar Döffinger]

On Thu, Sep 09, 2010 at 10:03:12PM +0100, M?ns Rullg?rd wrote:
> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> 
> > On Thu, Sep 09, 2010 at 10:03:39PM +0200, Michael Niedermayer wrote:
> >> On Thu, Sep 09, 2010 at 01:00:59AM +0100, irc at mansr.com wrote:
> >> [...]
> >> > [11:05:50] <cartman> libmpcodecs/vf_yadif.c:262:9: error: ambiguous instructions require an explicit suffix (could be 'cmpb', 'cmpw', 'cmpl',
> >> > [11:05:54] <cartman>       or 'cmpq')
> >> > [11:05:59] <cartman> somebody needs to tell michael
> >> > [11:07:03] * funman thinks somebody should be cartman
> >> > [11:07:46] <cartman> funman: he would flame me to hell
> >> > [11:07:47] <cartman> :P
> >> 
> >> do you mean this line:
> >>     19836     lorenm             "cmp       $2, %[mode] \n\t"\
> >> ?
> >
> > Hm, I don't think this can work on a 64 bit system.
> > I guess it would be possible to change the constraint to "m" and
> > use cmpl, or use different code for 32 vs. 64 bit, but it all
> > seems ugly to me.
> 
> The obvious solution is to move the code to yasm.

That's silly, yasm will not automatically switch between register
and memory storage, if you don't care about that feature you
can just replace "g" by either "r" or "m" and you don't have that
issue either.