[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 03/11] disas/nios2: Simplify endianess conversion
From: |
Laurent Vivier |
Subject: |
[PULL 03/11] disas/nios2: Simplify endianess conversion |
Date: |
Sat, 23 Oct 2021 20:31:15 +0200 |
From: Philippe Mathieu-Daudé <f4bug@amsat.org>
Since commit 12b6e9b27d4 ("disas: Clean up CPUDebug initialization")
the disassemble_info->bfd_endian enum is set for all targets in
target_disas(). We can directly call print_insn_nios2() and simplify.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20210807110939.95853-3-f4bug@amsat.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
---
disas/nios2.c | 22 +++-------------------
include/disas/dis-asm.h | 3 +--
target/nios2/cpu.c | 6 +-----
3 files changed, 5 insertions(+), 26 deletions(-)
diff --git a/disas/nios2.c b/disas/nios2.c
index d124902ae3e1..98ac07d72e9d 100644
--- a/disas/nios2.c
+++ b/disas/nios2.c
@@ -3478,9 +3478,7 @@ nios2_disassemble (bfd_vma address, unsigned long opcode,
instruction word at the address given, and prints the disassembled
instruction on the stream info->stream using info->fprintf_func. */
-static int
-print_insn_nios2 (bfd_vma address, disassemble_info *info,
- enum bfd_endian endianness)
+int print_insn_nios2(bfd_vma address, disassemble_info *info)
{
bfd_byte buffer[INSNLEN];
int status;
@@ -3488,7 +3486,7 @@ print_insn_nios2 (bfd_vma address, disassemble_info *info,
status = (*info->read_memory_func)(address, buffer, INSNLEN, info);
if (status == 0) {
unsigned long insn;
- if (endianness == BFD_ENDIAN_BIG) {
+ if (info->endian == BFD_ENDIAN_BIG) {
insn = (unsigned long) bfd_getb32(buffer);
} else {
insn = (unsigned long) bfd_getl32(buffer);
@@ -3501,7 +3499,7 @@ print_insn_nios2 (bfd_vma address, disassemble_info *info,
status = (*info->read_memory_func)(address, buffer, 2, info);
if (status == 0) {
unsigned long insn;
- if (endianness == BFD_ENDIAN_BIG) {
+ if (info->endian == BFD_ENDIAN_BIG) {
insn = (unsigned long) bfd_getb16(buffer);
} else {
insn = (unsigned long) bfd_getl16(buffer);
@@ -3514,17 +3512,3 @@ print_insn_nios2 (bfd_vma address, disassemble_info
*info,
(*info->memory_error_func)(status, address, info);
return -1;
}
-
-/* These two functions are the main entry points, accessed from
- disassemble.c. */
-int
-print_insn_big_nios2 (bfd_vma address, disassemble_info *info)
-{
- return print_insn_nios2 (address, info, BFD_ENDIAN_BIG);
-}
-
-int
-print_insn_little_nios2 (bfd_vma address, disassemble_info *info)
-{
- return print_insn_nios2 (address, info, BFD_ENDIAN_LITTLE);
-}
diff --git a/include/disas/dis-asm.h b/include/disas/dis-asm.h
index 524f29196d9d..08e1beec854f 100644
--- a/include/disas/dis-asm.h
+++ b/include/disas/dis-asm.h
@@ -455,8 +455,7 @@ int print_insn_crisv32 (bfd_vma,
disassemble_info*);
int print_insn_crisv10 (bfd_vma, disassemble_info*);
int print_insn_microblaze (bfd_vma, disassemble_info*);
int print_insn_ia64 (bfd_vma, disassemble_info*);
-int print_insn_big_nios2 (bfd_vma, disassemble_info*);
-int print_insn_little_nios2 (bfd_vma, disassemble_info*);
+int print_insn_nios2(bfd_vma, disassemble_info*);
int print_insn_xtensa (bfd_vma, disassemble_info*);
int print_insn_riscv32 (bfd_vma, disassemble_info*);
int print_insn_riscv64 (bfd_vma, disassemble_info*);
diff --git a/target/nios2/cpu.c b/target/nios2/cpu.c
index 947bb09bc1ed..58ecd27d757e 100644
--- a/target/nios2/cpu.c
+++ b/target/nios2/cpu.c
@@ -147,11 +147,7 @@ static void nios2_cpu_disas_set_info(CPUState *cpu,
disassemble_info *info)
{
/* NOTE: NiosII R2 is not supported yet. */
info->mach = bfd_arch_nios2;
-#ifdef TARGET_WORDS_BIGENDIAN
- info->print_insn = print_insn_big_nios2;
-#else
- info->print_insn = print_insn_little_nios2;
-#endif
+ info->print_insn = print_insn_nios2;
}
static int nios2_cpu_gdb_read_register(CPUState *cs, GByteArray *mem_buf, int
n)
--
2.31.1
- [PULL 00/11] Trivial branch for 6.2 patches, Laurent Vivier, 2021/10/23
- [PULL 01/11] po: update turkish translation, Laurent Vivier, 2021/10/23
- [PULL 04/11] MAINTAINERS: Add myself as reviewer of 'Machine core' API, Laurent Vivier, 2021/10/23
- [PULL 05/11] softmmu/physmem.c: Fix typo in comment, Laurent Vivier, 2021/10/23
- [PULL 11/11] analyze-migration.py: fix extract contents ('-x') errors, Laurent Vivier, 2021/10/23
- [PULL 10/11] analyze-migration.py: fix a long standing typo, Laurent Vivier, 2021/10/23
- [PULL 09/11] README: Fix some documentation URLs, Laurent Vivier, 2021/10/23
- [PULL 03/11] disas/nios2: Simplify endianess conversion,
Laurent Vivier <=
- [PULL 07/11] hw/nvram: Fix Memory Leak in Xilinx Versal eFuse device, Laurent Vivier, 2021/10/23
- [PULL 02/11] disas/nios2: Fix style in print_insn_nios2(), Laurent Vivier, 2021/10/23
- [PULL 06/11] hw/nvram: Fix Memory Leak in Xilinx eFuse QOM, Laurent Vivier, 2021/10/23
- [PULL 08/11] hw/nvram: Fix Memory Leak in Xilinx ZynqMP eFuse device, Laurent Vivier, 2021/10/23
- Re: [PULL 00/11] Trivial branch for 6.2 patches, Richard Henderson, 2021/10/23