On 2012-07-01 14:16:55 +0100, Måns Rullgård wrote:
> Janne Grunau <[email protected]> writes:
> 
> > On 2012-07-01 13:08:47 +0100, Måns Rullgård wrote:
> >>  #if HAVE_MMX
> >> +static void gradfun_filter_line_mmx2(uint8_t *dst, uint8_t *src, uint16_t 
> >> *dc, int width, int thresh, const uint16_t *dithers)
> >
> > we have HAVE_MMX2 if you like change this inconsistency. Or we could
> > remove the mmx2 check in configure since there aren't many systems
> > around which can't assemble mmx2 instructions
> 
> Does the function require mmx2?  I prefer not to make any assumptions
> when fixing this kind of things.

yes, 'pmaxsw' is a SSE instruction and part of AMD's mmx extension

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

Reply via email to