From 96c7115a9a1669c70331f594eca40eebeb5f2d2c Mon Sep 17 00:00:00 2001 From: Marcus Nilsson Date: Thu, 2 Dec 2021 13:57:11 +0000 Subject: [PATCH] Allow the --visualize-jumps feature to work with the AVR disassembler. * avr-dis.c (avr_operand); Pass in disassemble_info and fill in insn_type on branching instructions. --- opcodes/ChangeLog | 5 +++++ opcodes/avr-dis.c | 37 ++++++++++++++++++++++++++++++++----- 2 files changed, 37 insertions(+), 5 deletions(-) diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 7bf0094cf0d..7eba5165637 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,8 @@ +2021-12-02 Marcus Nilsson + + * avr-dis.c (avr_operand); Pass in disassemble_info and fill + in insn_type on branching instructions. + 2021-11-25 Andrew Burgess Simon Cook diff --git a/opcodes/avr-dis.c b/opcodes/avr-dis.c index 06f8a0c663b..61af515e73e 100644 --- a/opcodes/avr-dis.c +++ b/opcodes/avr-dis.c @@ -49,8 +49,17 @@ const struct avr_opcodes_s avr_opcodes[] = static const char * comment_start = "0x"; static int -avr_operand (unsigned int insn, unsigned int insn2, unsigned int pc, int constraint, - char *opcode_str, char *buf, char *comment, int regs, int *sym, bfd_vma *sym_addr) +avr_operand (unsigned int insn, + unsigned int insn2, + unsigned int pc, + int constraint, + char * opcode_str, + char * buf, + char * comment, + int regs, + int * sym, + bfd_vma * sym_addr, + disassemble_info * info) { int ok = 1; *sym = 0; @@ -161,6 +170,9 @@ avr_operand (unsigned int insn, unsigned int insn2, unsigned int pc, int constra objdump_print_address() which would affect many targets. */ sprintf (buf, "%#lx", (unsigned long) *sym_addr); strcpy (comment, comment_start); + info->insn_info_valid = 1; + info->insn_type = dis_jsr; + info->target = *sym_addr; break; case 'L': @@ -170,6 +182,9 @@ avr_operand (unsigned int insn, unsigned int insn2, unsigned int pc, int constra *sym = 1; *sym_addr = pc + 2 + rel_addr; strcpy (comment, comment_start); + info->insn_info_valid = 1; + info->insn_type = dis_branch; + info->target = *sym_addr; } break; @@ -181,6 +196,9 @@ avr_operand (unsigned int insn, unsigned int insn2, unsigned int pc, int constra *sym = 1; *sym_addr = pc + 2 + rel_addr; strcpy (comment, comment_start); + info->insn_info_valid = 1; + info->insn_type = dis_condbranch; + info->target = *sym_addr; } break; @@ -314,6 +332,13 @@ print_insn_avr (bfd_vma addr, disassemble_info *info) int sym_op1 = 0, sym_op2 = 0; bfd_vma sym_addr1, sym_addr2; + /* Clear instruction information field. */ + info->insn_info_valid = 0; + info->branch_delay_insns = 0; + info->data_size = 0; + info->insn_type = dis_noninsn; + info->target = 0; + info->target2 = 0; if (!initialized) { @@ -395,11 +420,13 @@ print_insn_avr (bfd_vma addr, disassemble_info *info) { int regs = REGISTER_P (*constraints); - ok = avr_operand (insn, insn2, addr, *constraints, opcode_str, op1, comment1, 0, &sym_op1, &sym_addr1); + ok = avr_operand (insn, insn2, addr, *constraints, opcode_str, op1, + comment1, 0, &sym_op1, &sym_addr1, info); if (ok && *(++constraints) == ',') - ok = avr_operand (insn, insn2, addr, *(++constraints), opcode_str, op2, - *comment1 ? comment2 : comment1, regs, &sym_op2, &sym_addr2); + ok = avr_operand (insn, insn2, addr, *(++constraints), opcode_str, + op2, *comment1 ? comment2 : comment1, regs, + &sym_op2, &sym_addr2, info); } } -- 2.30.2