On 01/14/2013 04:29 PM, Diego Biurrun wrote:
> ---
> 
> Added macro argument comments as requested by Justin.
> 
>  libavutil/x86/x86util.asm |   14 ++++++--------
>  1 files changed, 6 insertions(+), 8 deletions(-)
> 
> diff --git a/libavutil/x86/x86util.asm b/libavutil/x86/x86util.asm
> index 4740b7f..9236991 100644
> --- a/libavutil/x86/x86util.asm
> +++ b/libavutil/x86/x86util.asm
> @@ -221,13 +221,18 @@
>  %endif
>  %endmacro
>  
> -%macro ABSB2_MMX 4
> +%macro ABSB2 4 ; src1, src2, tmp1, tmp2 (tmp1/2 unused for SSSE3)
> +%if cpuflag(ssse3)
> +    pabsb   %1, %1
> +    pabsb   %2, %2
> +%else
>      pxor    %3, %3
>      pxor    %4, %4
>      psubb   %3, %1
>      psubb   %4, %2
>      pminub  %1, %3
>      pminub  %2, %4
> +%endif
>  %endmacro
>  
>  %macro ABSD2_MMX 4
> @@ -241,18 +246,11 @@
>      psubd   %2, %4
>  %endmacro
>  
> -%macro ABSB2_SSSE3 4
> -    pabsb   %1, %1
> -    pabsb   %2, %2
> -%endmacro
> -
>  %macro ABS4 6
>      ABS2 %1, %2, %5, %6
>      ABS2 %3, %4, %5, %6
>  %endmacro
>  
> -%define ABSB2 ABSB2_MMX
> -
>  %macro SPLATB_LOAD 3
>  %if cpuflag(ssse3)
>      movd      %1, [%2-3]

LGTM

-Justin

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

Reply via email to