[PATCH 4/4] scripts/generate_iga64_codes: remove parenthesis around ARG macro body

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Mar 20 17:02:38 UTC 2025


Hi Andrzej,
On 2025-03-14 at 15:59:11 +0100, Andrzej Hajda wrote:
> Parenthesis around macro body were added to follow C conventions.
> For newer platforms they are not allowed in such context.
> For iga64 they do not change anything and can be omited.
> 
> Signed-off-by: Andrzej Hajda <andrzej.hajda at intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  scripts/generate_iga64_codes | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/generate_iga64_codes b/scripts/generate_iga64_codes
> index bfef0e5c2853..1675ffe639b3 100755
> --- a/scripts/generate_iga64_codes
> +++ b/scripts/generate_iga64_codes
> @@ -92,7 +92,7 @@ EOF
>  # Compiles assembly to binary representation sent to stdout.
>  compile_iga64() {
>      cmd="cpp -P - -o $WD/$asm_name.$gen_name.asm"
> -    cmd+=" -DGEN_VER=$gen_ver -D'ARG(n)=($IGA64_ARG0 + (n))' -imacros $IGT_SRCDIR/lib/iga64_macros.h"
> +    cmd+=" -DGEN_VER=$gen_ver -D'ARG(n)=$IGA64_ARG0 + n' -imacros $IGT_SRCDIR/lib/iga64_macros.h"
>      eval "$cmd" <<<"$asm_body" || die "cpp error for $asm_name.$gen_name\ncmd: $cmd"
>      cmd="iga64 -Xauto-deps -Wall -p=$gen_name"
>      cmd+=" $WD/$asm_name.$gen_name.asm"
> 
> -- 
> 2.34.1
> 


More information about the igt-dev mailing list