On Thu, Jul 7, 2011 at 5:33 PM, Ronald S. Bultje <[email protected]> wrote:

> Hi,
>
> On Thu, Jul 7, 2011 at 2:24 PM, Daniel Kang <[email protected]>
> wrote:
> [..]
> >
> +;-----------------------------------------------------------------------------
> > +; void pred16x16_vertical(pixel *src, int stride)
> >
> +;-----------------------------------------------------------------------------
> > +%macro MOV16 3-5
> > +%if mmsize==16
> > +    movdqa [%1+ 0], %2
> > +    movdqa [%1+16], %3
> > +%else
> > +    movq   [%1+ 0], %2
> > +    movq   [%1+ 8], %3
> > +    movq   [%1+16], %4
> > +    movq   [%1+24], %5
> > +%endif
> > +%endmacro
>
> %macro MOV16 3-5
> mova [%1+0], %2
> mova [%1+mmsize], %3
> %if mmsize == 8
> mova [%1+16], %4
> mova [%1+24], %5
> %endif
> %endmacro
>
> Lot of code in here, but looks pretty good. No major complaints from me.


Awesome. Fixed, and thanks for the review.

Attachment: patch.diff
Description: Binary data

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

Reply via email to