diff mbox series

[v3,14/27] disas/microblaze: Split out print_immval_addr

Message ID 20240412073346.458116-15-richard.henderson@linaro.org
State Superseded
Headers show
Series misc: Replace sprintf | expand

Commit Message

Richard Henderson April 12, 2024, 7:33 a.m. UTC
Unify the code blocks that try to print a symbolic address.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 disas/microblaze.c | 89 +++++++++++++++-------------------------------
 1 file changed, 29 insertions(+), 60 deletions(-)

Comments

Pierrick Bouvier June 3, 2024, 7:14 p.m. UTC | #1
On 4/12/24 00:33, Richard Henderson wrote:
> Unify the code blocks that try to print a symbolic address.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   disas/microblaze.c | 89 +++++++++++++++-------------------------------
>   1 file changed, 29 insertions(+), 60 deletions(-)
> 
> diff --git a/disas/microblaze.c b/disas/microblaze.c
> index 49a4c0fd40..3473c94164 100644
> --- a/disas/microblaze.c
> +++ b/disas/microblaze.c
> @@ -767,6 +767,24 @@ read_insn_microblaze (bfd_vma memaddr,
>     return inst;
>   }
>   
> +static void print_immval_addr(struct disassemble_info *info, bool immfound,
> +                              int immval, unsigned inst, int addend)
> +{
> +    if (info->print_address_func && info->symbol_at_address_func) {
> +        if (immfound) {
> +            immval |= get_int_field_imm(inst) & 0x0000ffff;
> +        } else {
> +            immval = (int16_t)get_int_field_imm(inst);
> +        }
> +        immval += addend;
> +        if (immval != 0 && info->symbol_at_address_func(immval, info)) {
> +            info->fprintf_func(info->stream, "\t// ");
> +            info->print_address_func (immval, info);
> +        } else if (addend) {
> +            info->fprintf_func(info->stream, "\t// %x", immval);
> +        }
> +    }
> +}
>   
>   int
>   print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
> @@ -821,18 +839,8 @@ print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
>        break;
>           case INST_TYPE_RD_R1_IMM:
>   	  fprintf_func(stream, "\t%s, %s, %s", get_field_rd(inst), get_field_r1(inst), get_field_imm(inst));
> -	  if (info->print_address_func && get_int_field_r1(inst) == 0 && info->symbol_at_address_func) {
> -	    if (immfound)
> -	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
> -	    else {
> -	      immval = get_int_field_imm(inst);
> -	      if (immval & 0x8000)
> -		immval |= 0xFFFF0000;
> -	    }
> -	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
> -	      fprintf_func (stream, "\t// ");
> -	      info->print_address_func (immval, info);
> -	    }
> +	  if (get_int_field_r1(inst) == 0) {
> +              print_immval_addr(info, immfound, immval, inst, 0);
>   	  }
>   	  break;
>   	case INST_TYPE_RD_R1_IMM5:
> @@ -860,61 +868,22 @@ print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
>   	  fprintf_func(stream, "\t%s, %s", get_field_r1(inst), get_field_imm(inst));
>   	  /* The non-pc relative instructions are returns, which shouldn't
>   	     have a label printed */
> -	  if (info->print_address_func && op->inst_offset_type == INST_PC_OFFSET && info->symbol_at_address_func) {
> -	    if (immfound)
> -	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
> -	    else {
> -	      immval = get_int_field_imm(inst);
> -	      if (immval & 0x8000)
> -		immval |= 0xFFFF0000;
> -	    }
> -	    immval += memaddr;
> -	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
> -	      fprintf_func (stream, "\t// ");
> -	      info->print_address_func (immval, info);
> -	    } else {
> -	      fprintf_func (stream, "\t\t// ");
> -	      fprintf_func (stream, "%x", immval);
> -	    }
> +	  if (op->inst_offset_type == INST_PC_OFFSET) {
> +              print_immval_addr(info, immfound, immval, inst, memaddr);
>   	  }
>   	  break;
>           case INST_TYPE_RD_IMM:
>   	  fprintf_func(stream, "\t%s, %s", get_field_rd(inst), get_field_imm(inst));
> -	  if (info->print_address_func && info->symbol_at_address_func) {
> -	    if (immfound)
> -	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
> -	    else {
> -	      immval = get_int_field_imm(inst);
> -	      if (immval & 0x8000)
> -		immval |= 0xFFFF0000;
> -	    }
> -	    if (op->inst_offset_type == INST_PC_OFFSET)
> -	      immval += (int) memaddr;
> -	    if (info->symbol_at_address_func(immval, info)) {
> -	      fprintf_func (stream, "\t// ");
> -	      info->print_address_func (immval, info);
> -	    }
> -	  }
> +          print_immval_addr(info, immfound, immval, inst,
> +                            op->inst_offset_type == INST_PC_OFFSET
> +                            ? memaddr : 0);
>   	  break;
>           case INST_TYPE_IMM:
>   	  fprintf_func(stream, "\t%s", get_field_imm(inst));
> -	  if (info->print_address_func && info->symbol_at_address_func && op->instr != imm) {
> -	    if (immfound)
> -	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
> -	    else {
> -	      immval = get_int_field_imm(inst);
> -	      if (immval & 0x8000)
> -		immval |= 0xFFFF0000;
> -	    }
> -	    if (op->inst_offset_type == INST_PC_OFFSET)
> -	      immval += (int) memaddr;
> -	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
> -	      fprintf_func (stream, "\t// ");
> -	      info->print_address_func (immval, info);
> -	    } else if (op->inst_offset_type == INST_PC_OFFSET) {
> -	      fprintf_func (stream, "\t\t// ");
> -	      fprintf_func (stream, "%x", immval);
> -	    }
> +	  if (op->instr != imm) {
> +              print_immval_addr(info, immfound, immval, inst,
> +                                op->inst_offset_type == INST_PC_OFFSET
> +                                ? memaddr : 0);
>   	  }
>   	  break;
>           case INST_TYPE_RD_R2:

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
diff mbox series

Patch

diff --git a/disas/microblaze.c b/disas/microblaze.c
index 49a4c0fd40..3473c94164 100644
--- a/disas/microblaze.c
+++ b/disas/microblaze.c
@@ -767,6 +767,24 @@  read_insn_microblaze (bfd_vma memaddr,
   return inst;
 }
 
+static void print_immval_addr(struct disassemble_info *info, bool immfound,
+                              int immval, unsigned inst, int addend)
+{
+    if (info->print_address_func && info->symbol_at_address_func) {
+        if (immfound) {
+            immval |= get_int_field_imm(inst) & 0x0000ffff;
+        } else {
+            immval = (int16_t)get_int_field_imm(inst);
+        }
+        immval += addend;
+        if (immval != 0 && info->symbol_at_address_func(immval, info)) {
+            info->fprintf_func(info->stream, "\t// ");
+            info->print_address_func (immval, info);
+        } else if (addend) {
+            info->fprintf_func(info->stream, "\t// %x", immval);
+        }
+    }
+}
 
 int 
 print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
@@ -821,18 +839,8 @@  print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
      break;
         case INST_TYPE_RD_R1_IMM:
 	  fprintf_func(stream, "\t%s, %s, %s", get_field_rd(inst), get_field_r1(inst), get_field_imm(inst));
-	  if (info->print_address_func && get_int_field_r1(inst) == 0 && info->symbol_at_address_func) {
-	    if (immfound)
-	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
-	    else {
-	      immval = get_int_field_imm(inst);
-	      if (immval & 0x8000)
-		immval |= 0xFFFF0000;
-	    }
-	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
-	      fprintf_func (stream, "\t// ");
-	      info->print_address_func (immval, info);
-	    }
+	  if (get_int_field_r1(inst) == 0) {
+              print_immval_addr(info, immfound, immval, inst, 0);
 	  }
 	  break;
 	case INST_TYPE_RD_R1_IMM5:
@@ -860,61 +868,22 @@  print_insn_microblaze (bfd_vma memaddr, struct disassemble_info * info)
 	  fprintf_func(stream, "\t%s, %s", get_field_r1(inst), get_field_imm(inst));
 	  /* The non-pc relative instructions are returns, which shouldn't 
 	     have a label printed */
-	  if (info->print_address_func && op->inst_offset_type == INST_PC_OFFSET && info->symbol_at_address_func) {
-	    if (immfound)
-	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
-	    else {
-	      immval = get_int_field_imm(inst);
-	      if (immval & 0x8000)
-		immval |= 0xFFFF0000;
-	    }
-	    immval += memaddr;
-	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
-	      fprintf_func (stream, "\t// ");
-	      info->print_address_func (immval, info);
-	    } else {
-	      fprintf_func (stream, "\t\t// ");
-	      fprintf_func (stream, "%x", immval);
-	    }
+	  if (op->inst_offset_type == INST_PC_OFFSET) {
+              print_immval_addr(info, immfound, immval, inst, memaddr);
 	  }
 	  break;
         case INST_TYPE_RD_IMM:
 	  fprintf_func(stream, "\t%s, %s", get_field_rd(inst), get_field_imm(inst));
-	  if (info->print_address_func && info->symbol_at_address_func) {
-	    if (immfound)
-	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
-	    else {
-	      immval = get_int_field_imm(inst);
-	      if (immval & 0x8000)
-		immval |= 0xFFFF0000;
-	    }
-	    if (op->inst_offset_type == INST_PC_OFFSET)
-	      immval += (int) memaddr;
-	    if (info->symbol_at_address_func(immval, info)) {
-	      fprintf_func (stream, "\t// ");
-	      info->print_address_func (immval, info);
-	    } 
-	  }
+          print_immval_addr(info, immfound, immval, inst,
+                            op->inst_offset_type == INST_PC_OFFSET
+                            ? memaddr : 0);
 	  break;
         case INST_TYPE_IMM:
 	  fprintf_func(stream, "\t%s", get_field_imm(inst));
-	  if (info->print_address_func && info->symbol_at_address_func && op->instr != imm) {
-	    if (immfound)
-	      immval |= (get_int_field_imm(inst) & 0x0000ffff);
-	    else {
-	      immval = get_int_field_imm(inst);
-	      if (immval & 0x8000)
-		immval |= 0xFFFF0000;
-	    }
-	    if (op->inst_offset_type == INST_PC_OFFSET)
-	      immval += (int) memaddr;
-	    if (immval > 0 && info->symbol_at_address_func(immval, info)) {
-	      fprintf_func (stream, "\t// ");
-	      info->print_address_func (immval, info);
-	    } else if (op->inst_offset_type == INST_PC_OFFSET) {
-	      fprintf_func (stream, "\t\t// ");
-	      fprintf_func (stream, "%x", immval);
-	    }
+	  if (op->instr != imm) {
+              print_immval_addr(info, immfound, immval, inst,
+                                op->inst_offset_type == INST_PC_OFFSET
+                                ? memaddr : 0);
 	  }
 	  break;
         case INST_TYPE_RD_R2: