On 03/06/2018 05:43 PM, Michael Clark wrote: > This was added to help debug issues using -d in_asm. It is > useful to see the instruction bytes, as one can detect if > one is trying to execute ASCII or device-tree magic.
clean :) > > Signed-off-by: Michael Clark <m...@sifive.com> > Signed-off-by: Palmer Dabbelt <pal...@sifive.com> Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > disas/riscv.c | 39 ++++++++++++++++++++------------------- > 1 file changed, 20 insertions(+), 19 deletions(-) > > diff --git a/disas/riscv.c b/disas/riscv.c > index 3c17501..4580308 100644 > --- a/disas/riscv.c > +++ b/disas/riscv.c > @@ -2769,25 +2769,6 @@ static void format_inst(char *buf, size_t buflen, > size_t tab, rv_decode *dec) > char tmp[64]; > const char *fmt; > > - if (dec->op == rv_op_illegal) { > - size_t len = inst_length(dec->inst); > - switch (len) { > - case 2: > - snprintf(buf, buflen, "(0x%04" PRIx64 ")", dec->inst); > - break; > - case 4: > - snprintf(buf, buflen, "(0x%08" PRIx64 ")", dec->inst); > - break; > - case 6: > - snprintf(buf, buflen, "(0x%012" PRIx64 ")", dec->inst); > - break; > - default: > - snprintf(buf, buflen, "(0x%016" PRIx64 ")", dec->inst); > - break; > - } > - return; > - } > - > fmt = opcode_data[dec->op].format; > while (*fmt) { > switch (*fmt) { > @@ -3004,6 +2985,11 @@ disasm_inst(char *buf, size_t buflen, rv_isa isa, > uint64_t pc, rv_inst inst) > format_inst(buf, buflen, 16, &dec); > } > > +#define INST_FMT_2 "%04" PRIx64 " " > +#define INST_FMT_4 "%08" PRIx64 " " > +#define INST_FMT_6 "%012" PRIx64 " " > +#define INST_FMT_8 "%016" PRIx64 " " > + > static int > print_insn_riscv(bfd_vma memaddr, struct disassemble_info *info, rv_isa isa) > { > @@ -3031,6 +3017,21 @@ print_insn_riscv(bfd_vma memaddr, struct > disassemble_info *info, rv_isa isa) > } > } > > + switch (len) { > + case 2: > + (*info->fprintf_func)(info->stream, INST_FMT_2, inst); > + break; > + case 4: > + (*info->fprintf_func)(info->stream, INST_FMT_4, inst); > + break; > + case 6: > + (*info->fprintf_func)(info->stream, INST_FMT_6, inst); > + break; > + default: > + (*info->fprintf_func)(info->stream, INST_FMT_8, inst); > + break; > + } > + > disasm_inst(buf, sizeof(buf), isa, memaddr, inst); > (*info->fprintf_func)(info->stream, "%s", buf); > >