> +INIT_XMM sse2
> +cglobal conv_fltp_to_s16_2ch, 3,4,3, dst, src0, len, src1
> +    lea      lenq, [4*lend]
> +    mov     src1q, [src0q+gprsize]
> +    mov     src0q, [src0q        ]
> +    add      dstq, lenq
> +    add     src0q, lenq
> +    add     src1q, lenq
> +    neg      lenq
> +    mova       m2, [pf_s16_scale]
> +    ALIGN 16
> +.loop:
> +    mulps      m0, m2, [src0q+lenq]
> +    mulps      m1, m2, [src1q+lenq]

Same comment on mulps as in other emails, and same note on "why isn't
there an AVX version?"

> +    cvtps2dq   m0, m0
> +    cvtps2dq   m1, m1
> +    packssdw   m0, m1
> +    movhlps    m1, m0
> +    punpcklwd  m0, m1
> +    mova  [dstq+lenq], m0
> +    add      lenq, mmsize
> +    js .loop
> +    REP_RET

Otherwise looks okay.

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

Reply via email to