Re: [FFmpeg-devel] [PATCH] aarch64: hevcdsp_idct: Reuse preexisting macros for transposes

["Martin Storsjö" <martin@martin.st>]

On Thu, 23 Feb 2023, Martin Storsjö wrote:

> Signed-off-by: Martin Storsjö <martin@martin.st>
> ---
> libavcodec/aarch64/hevcdsp_idct_neon.S | 16 +++-------------
> 1 file changed, 3 insertions(+), 13 deletions(-)
>
> diff --git a/libavcodec/aarch64/hevcdsp_idct_neon.S b/libavcodec/aarch64/hevcdsp_idct_neon.S
> index 124c50998a..c61fcb175d 100644
> --- a/libavcodec/aarch64/hevcdsp_idct_neon.S
> +++ b/libavcodec/aarch64/hevcdsp_idct_neon.S
> @@ -25,6 +25,7 @@
>  */
>
> #include "libavutil/aarch64/asm.S"
> +#include "neon.S"
>
> const trans, align=4
>         .short          64, 83, 64, 36
> @@ -279,20 +280,9 @@ endfunc
>          sub            \out3, v28.4s, v30.4s
> .endm
>
> -.macro transpose8_4x4 r0, r1, r2, r3
> -        trn1            v2.8h, \r0\().8h, \r1\().8h
> -        trn2            v3.8h, \r0\().8h, \r1\().8h
> -        trn1            v4.8h, \r2\().8h, \r3\().8h
> -        trn2            v5.8h, \r2\().8h, \r3\().8h
> -        trn1            \r0\().4s, v2.4s, v4.4s
> -        trn2            \r2\().4s, v2.4s, v4.4s
> -        trn1            \r1\().4s, v3.4s, v5.4s
> -        trn2            \r3\().4s, v3.4s, v5.4s
> -.endm
> -
> .macro transpose_8x8 r0, r1, r2, r3, r4, r5, r6, r7
> -        transpose8_4x4  \r0, \r1, \r2, \r3
> -        transpose8_4x4  \r4, \r5, \r6, \r7
> +        transpose_4x8H  \r0, \r1, \r2, \r3, v2, v3, v4, v5
> +        transpose_4x8H  \r4, \r5, \r6, \r7, v2, v3, v4, v5
> .endm

I'll go ahead and push this.

// Martin
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".