[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