aboutsummaryrefslogtreecommitdiff
path: root/disas
diff options
context:
space:
mode:
authorRichard Henderson <richard.henderson@linaro.org>2024-04-12 00:33:35 -0700
committerRichard Henderson <richard.henderson@linaro.org>2024-06-05 12:29:54 -0700
commitde66f9f7ab215e09b0514dbdcd8450f42efdaa9a (patch)
treedc5b5466fb23cd6fd50bc13b41c13ac1a6492ba7 /disas
parentc3c6fed6460051e265f59d4cd451865faa3e8d71 (diff)
downloadqemu-de66f9f7ab215e09b0514dbdcd8450f42efdaa9a.zip
qemu-de66f9f7ab215e09b0514dbdcd8450f42efdaa9a.tar.gz
qemu-de66f9f7ab215e09b0514dbdcd8450f42efdaa9a.tar.bz2
disas/microblaze: Merge op->name output into each fprintf
In the common case, issue one single fprintf. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org> Message-Id: <20240412073346.458116-17-richard.henderson@linaro.org>
Diffstat (limited to 'disas')
-rw-r--r--disas/microblaze.c80
1 files changed, 40 insertions, 40 deletions
diff --git a/disas/microblaze.c b/disas/microblaze.c
index c729c76..a537ac6 100644
--- a/disas/microblaze.c
+++ b/disas/microblaze.c
@@ -830,54 +830,52 @@ print_insn_microblaze(bfd_vma memaddr, struct disassemble_info *info)
return 4;
}
- fprintf_func (stream, "%s", op->name);
-
switch (op->inst_type) {
case INST_TYPE_RD_R1_R2:
- fprintf_func(stream, "\t%s, %s, %s",
- get_field_rd(inst), get_field_r1(inst),
+ fprintf_func(stream, "%s\t%s, %s, %s",
+ op->name, get_field_rd(inst), get_field_r1(inst),
get_field_r2(inst));
break;
case INST_TYPE_RD_R1_IMM:
- fprintf_func(stream, "\t%s, %s, %s",
- get_field_rd(inst), get_field_r1(inst),
+ fprintf_func(stream, "%s\t%s, %s, %s",
+ op->name, get_field_rd(inst), get_field_r1(inst),
get_field_imm(inst));
if (get_int_field_r1(inst) == 0) {
print_immval_addr(info, immfound, immval, inst, 0);
}
break;
case INST_TYPE_RD_R1_IMM5:
- fprintf_func(stream, "\t%s, %s, %s",
- get_field_rd(inst), get_field_r1(inst),
+ fprintf_func(stream, "%s\t%s, %s, %s",
+ op->name, get_field_rd(inst), get_field_r1(inst),
get_field_imm5(inst));
break;
case INST_TYPE_RD_RFSL:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_rfsl(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_rfsl(inst));
break;
case INST_TYPE_R1_RFSL:
- fprintf_func(stream, "\t%s, %s",
- get_field_r1(inst), get_field_rfsl(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_r1(inst), get_field_rfsl(inst));
break;
case INST_TYPE_RD_SPECIAL:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_special(inst, op));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_special(inst, op));
break;
case INST_TYPE_SPECIAL_R1:
- fprintf_func(stream, "\t%s, %s",
- get_field_special(inst, op), get_field_r1(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_special(inst, op), get_field_r1(inst));
break;
case INST_TYPE_RD_R1:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_r1(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_r1(inst));
break;
case INST_TYPE_R1_R2:
- fprintf_func(stream, "\t%s, %s",
- get_field_r1(inst), get_field_r2(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_r1(inst), get_field_r2(inst));
break;
case INST_TYPE_R1_IMM:
- fprintf_func(stream, "\t%s, %s",
- get_field_r1(inst), get_field_imm(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_r1(inst), get_field_imm(inst));
/*
* The non-pc relative instructions are returns,
* which shouldn't have a label printed.
@@ -887,14 +885,15 @@ print_insn_microblaze(bfd_vma memaddr, struct disassemble_info *info)
}
break;
case INST_TYPE_RD_IMM:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_imm(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_imm(inst));
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));
+ fprintf_func(stream, "%s\t%s",
+ op->name, get_field_imm(inst));
if (op->instr != imm) {
print_immval_addr(info, immfound, immval, inst,
op->inst_offset_type == INST_PC_OFFSET
@@ -902,37 +901,38 @@ print_insn_microblaze(bfd_vma memaddr, struct disassemble_info *info)
}
break;
case INST_TYPE_RD_R2:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_r2(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_r2(inst));
break;
case INST_TYPE_R2:
- fprintf_func(stream, "\t%s",
- get_field_r2(inst));
+ fprintf_func(stream, "%s\t%s",
+ op->name, get_field_r2(inst));
break;
case INST_TYPE_R1:
- fprintf_func(stream, "\t%s",
- get_field_r1(inst));
+ fprintf_func(stream, "%s\t%s",
+ op->name, get_field_r1(inst));
break;
case INST_TYPE_RD_R1_SPECIAL:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_r2(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_r2(inst));
break;
case INST_TYPE_RD_IMM15:
- fprintf_func(stream, "\t%s, %s",
- get_field_rd(inst), get_field_imm15(inst));
+ fprintf_func(stream, "%s\t%s, %s",
+ op->name, get_field_rd(inst), get_field_imm15(inst));
break;
/* For tuqula instruction */
case INST_TYPE_RD:
- fprintf_func(stream, "\t%s",
- get_field_rd(inst));
+ fprintf_func(stream, "%s\t%s",
+ op->name, get_field_rd(inst));
break;
case INST_TYPE_RFSL:
- fprintf_func(stream, "\t%s",
- get_field_rfsl(inst));
+ fprintf_func(stream, "%s\t%s",
+ op->name, get_field_rfsl(inst));
break;
default:
/* if the disassembler lags the instruction set */
- fprintf_func(stream, "\tundecoded operands, inst is 0x%04lx", inst);
+ fprintf_func(stream, "%s\tundecoded operands, inst is 0x%04lx",
+ op->name, inst);
break;
}
return 4;