On Thu, Jul 26, 2012 at 08:54:30PM -0700, Ronald S. Bultje wrote:
> 
> --- a/libavcodec/x86/h264_idct_10bit.asm
> +++ b/libavcodec/x86/h264_idct_10bit.asm
> @@ -72,25 +72,25 @@ SECTION .text
>  
> -%macro ADD4x4IDCT 1
> -add4x4_idct_%1:
> +%macro ADD4x4IDCT 0
> +add4x4_idct_ %+ SUFFIX:

Drop the trailing '_', same below, but you already know that.

> @@ -107,28 +107,28 @@ add4x4_idct_%1:
>  
>  %macro ADD16_OP 3
>      cmp          byte [r4+%3], 0
>      jz .skipblock%2
>      mov         r5d, [r1+%2*4]
> -    call add4x4_idct_%1
> +    call add4x4_idct_ %+ SUFFIX
>  .skipblock%2:
>  %if %2<15
>      add          r2, 64
>  %endif
>  %endmacro

This macro now uses one less parameter, adjust it accordingly.

> -%macro IDCT_ADD16_10 1
> -cglobal h264_idct_add16_10_%1, 5,6
> +%macro IDCT_ADD16_10 0
> +cglobal h264_idct_add16_10, 5,6
>      ADD16_OP %1, 0, 4+1*8
>      ADD16_OP %1, 1, 5+1*8
>      ADD16_OP %1, 2, 4+2*8

By sheer luck, the macro kept working correctly :)

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

Reply via email to