Re: [PATCH] Powerpc: Add support for openat and fstatat syscalls

["Ulrich Weigand" <Ulrich.Weigand@de.ibm.com>]

"Carl Love" <cel@us.ibm.com> wrote on 13.10.2021 23:55:43:

> Powerpc: Add support for openat and fstatat syscalls
>
> [gdb] update ppc-linux-tdep.c
>
> Add argument to ppc_canonicalize_syscall for the wordsize.
> Add syscall entries for the openat and fstatat system calls.

> +  else if (syscall == 286)
> +    result = gdb_sys_openat;
> +  else if (syscall == 291)
> +    if (wordsize == 64)
> +      result = gdb_sys_newfstatat;
> +    else
> +      result = gdb_sys_fstatat64;

Please add braces around the nested if to avoid the
ambiguous else.

Otherwise, this is OK.

Thanks,
Ulrich