[FFmpeg-devel] [PATCH] lavc/aarch64/simple_idct: separate macro arguments with commas

Matthieu Bouron matthieu.bouron at gmail.com
Sun May 7 12:05:25 EEST 2017


Le 2 mai 2017 12:01 PM, "Benoit Fouet" <benoit.fouet at free.fr> a écrit :

Hi,


On 28/04/2017 21:58, Matthieu Bouron wrote:
> Untested: fixes ticket #6324.
> ---
>  libavcodec/aarch64/simple_idct_neon.S | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/libavcodec/aarch64/simple_idct_neon.S
b/libavcodec/aarch64/simple_idct_neon.S
> index 52273420f9..d31f72a609 100644
> --- a/libavcodec/aarch64/simple_idct_neon.S
> +++ b/libavcodec/aarch64/simple_idct_neon.S
> @@ -61,19 +61,19 @@ endconst
>          br              x10
>  .endm
>
> -.macro smull1 a b c
> +.macro smull1 a, b, c
>          smull           \a, \b, \c
>  .endm
>
> -.macro smlal1 a b c
> +.macro smlal1 a, b, c
>          smlal           \a, \b, \c
>  .endm
>
> -.macro smlsl1 a b c
> +.macro smlsl1 a, b, c
>          smlsl           \a, \b, \c
>  .endm
>
> -.macro idct_col4_top y1 y2 y3 y4 i l
> +.macro idct_col4_top y1, y2, y3, y4, i, l
>          smull\i         v7.4S,  \y3\().\l, z2
>          smull\i         v16.4S, \y3\().\l, z6
>          smull\i         v17.4S, \y2\().\l, z1
> @@ -91,7 +91,7 @@ endconst
>          smlsl\i         v6.4S,  \y4\().\l, z5
>  .endm
>
> -.macro idct_row4_neon y1 y2 y3 y4 pass
> +.macro idct_row4_neon y1, y2, y3, y4, pass
>          ld1             {\y1\().2D-\y2\().2D}, [x2], #32
>          movi            v23.4S, #1<<2, lsl #8
>          orr             v5.16B, \y1\().16B, \y2\().16B
> @@ -153,7 +153,7 @@ endconst
>          trn2            \y4\().4S, v17.4S, v19.4S
>  .endm
>
> -.macro declare_idct_col4_neon i l
> +.macro declare_idct_col4_neon i, l
>  function idct_col4_neon\i
>          dup             v23.4H, z4c
>  .if \i == 1

Sounds sane, but shouldn't we be doing this for all instances of
multiple arguments macros without commas?


Sure, I may have missed some. I will work again on this patch on Tuesday as
I will have access to an apple machine (and hopefully fix the build without
gas-preprocessor).

Sorry for the delay,
Matthieu


Thanks
--
Ben
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel at ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list