[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-6.2 1/2] disas/nios2: Fix style in print_insn_nios2()
From: |
Laurent Vivier |
Subject: |
Re: [PATCH-for-6.2 1/2] disas/nios2: Fix style in print_insn_nios2() |
Date: |
Wed, 29 Sep 2021 17:28:13 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 |
Le 07/08/2021 à 13:09, Philippe Mathieu-Daudé a écrit :
> We are going to modify this function, fix its style first.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> disas/nios2.c | 53 +++++++++++++++++++++++++--------------------------
> 1 file changed, 26 insertions(+), 27 deletions(-)
>
> diff --git a/disas/nios2.c b/disas/nios2.c
> index c3e82140c79..d124902ae3e 100644
> --- a/disas/nios2.c
> +++ b/disas/nios2.c
> @@ -3482,38 +3482,37 @@ static int
> print_insn_nios2 (bfd_vma address, disassemble_info *info,
> enum bfd_endian endianness)
> {
> - bfd_byte buffer[INSNLEN];
> - int status;
> + bfd_byte buffer[INSNLEN];
> + int status;
>
> - status = (*info->read_memory_func) (address, buffer, INSNLEN, info);
> - if (status == 0)
> - {
> - unsigned long insn;
> - if (endianness == BFD_ENDIAN_BIG)
> - insn = (unsigned long) bfd_getb32 (buffer);
> - else
> - insn = (unsigned long) bfd_getl32 (buffer);
> - return nios2_disassemble (address, insn, info);
> + status = (*info->read_memory_func)(address, buffer, INSNLEN, info);
> + if (status == 0) {
> + unsigned long insn;
> + if (endianness == BFD_ENDIAN_BIG) {
> + insn = (unsigned long) bfd_getb32(buffer);
> + } else {
> + insn = (unsigned long) bfd_getl32(buffer);
> + }
> + return nios2_disassemble(address, insn, info);
> }
>
> - /* We might have a 16-bit R2 instruction at the end of memory. Try that.
> */
> - if (info->mach == bfd_mach_nios2r2)
> - {
> - status = (*info->read_memory_func) (address, buffer, 2, info);
> - if (status == 0)
> - {
> - unsigned long insn;
> - if (endianness == BFD_ENDIAN_BIG)
> - insn = (unsigned long) bfd_getb16 (buffer);
> - else
> - insn = (unsigned long) bfd_getl16 (buffer);
> - return nios2_disassemble (address, insn, info);
> - }
> + /* We might have a 16-bit R2 instruction at the end of memory. Try that.
> */
> + if (info->mach == bfd_mach_nios2r2) {
> + status = (*info->read_memory_func)(address, buffer, 2, info);
> + if (status == 0) {
> + unsigned long insn;
> + if (endianness == BFD_ENDIAN_BIG) {
> + insn = (unsigned long) bfd_getb16(buffer);
> + } else {
> + insn = (unsigned long) bfd_getl16(buffer);
> + }
> + return nios2_disassemble(address, insn, info);
> + }
> }
>
> - /* If we got here, we couldn't read anything. */
> - (*info->memory_error_func) (status, address, info);
> - return -1;
> + /* If we got here, we couldn't read anything. */
> + (*info->memory_error_func)(status, address, info);
> + return -1;
> }
>
> /* These two functions are the main entry points, accessed from
>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH-for-6.2 1/2] disas/nios2: Fix style in print_insn_nios2(),
Laurent Vivier <=