[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 05/70] tcg/tci: Remove tci_read_r8
From: |
Richard Henderson |
Subject: |
[PATCH v3 05/70] tcg/tci: Remove tci_read_r8 |
Date: |
Sun, 7 Feb 2021 18:36:47 -0800 |
Use explicit casts for ext8u opcodes, and allow truncation
to happen with the store for st8 opcodes.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
tcg/tci.c | 23 +++++------------------
1 file changed, 5 insertions(+), 18 deletions(-)
diff --git a/tcg/tci.c b/tcg/tci.c
index fb3c97aaf1..c44a4aec7b 100644
--- a/tcg/tci.c
+++ b/tcg/tci.c
@@ -78,11 +78,6 @@ static int32_t tci_read_reg32s(const tcg_target_ulong *regs,
TCGReg index)
}
#endif
-static uint8_t tci_read_reg8(const tcg_target_ulong *regs, TCGReg index)
-{
- return (uint8_t)tci_read_reg(regs, index);
-}
-
static uint16_t tci_read_reg16(const tcg_target_ulong *regs, TCGReg index)
{
return (uint16_t)tci_read_reg(regs, index);
@@ -169,14 +164,6 @@ tci_read_r(const tcg_target_ulong *regs, const uint8_t
**tb_ptr)
return value;
}
-/* Read indexed register (8 bit) from bytecode. */
-static uint8_t tci_read_r8(const tcg_target_ulong *regs, const uint8_t
**tb_ptr)
-{
- uint8_t value = tci_read_reg8(regs, **tb_ptr);
- *tb_ptr += 1;
- return value;
-}
-
#if TCG_TARGET_HAS_ext8s_i32 || TCG_TARGET_HAS_ext8s_i64
/* Read indexed register (8 bit signed) from bytecode. */
static int8_t tci_read_r8s(const tcg_target_ulong *regs, const uint8_t
**tb_ptr)
@@ -550,7 +537,7 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState
*env,
tci_write_reg(regs, t0, *(uint32_t *)(t1 + t2));
break;
CASE_32_64(st8)
- t0 = tci_read_r8(regs, &tb_ptr);
+ t0 = tci_read_r(regs, &tb_ptr);
t1 = tci_read_r(regs, &tb_ptr);
t2 = tci_read_s32(&tb_ptr);
*(uint8_t *)(t1 + t2) = t0;
@@ -739,8 +726,8 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState
*env,
#if TCG_TARGET_HAS_ext8u_i32
case INDEX_op_ext8u_i32:
t0 = *tb_ptr++;
- t1 = tci_read_r8(regs, &tb_ptr);
- tci_write_reg(regs, t0, t1);
+ t1 = tci_read_r(regs, &tb_ptr);
+ tci_write_reg(regs, t0, (uint8_t)t1);
break;
#endif
#if TCG_TARGET_HAS_ext16u_i32
@@ -933,8 +920,8 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState
*env,
#if TCG_TARGET_HAS_ext8u_i64
case INDEX_op_ext8u_i64:
t0 = *tb_ptr++;
- t1 = tci_read_r8(regs, &tb_ptr);
- tci_write_reg(regs, t0, t1);
+ t1 = tci_read_r(regs, &tb_ptr);
+ tci_write_reg(regs, t0, (uint8_t)t1);
break;
#endif
#if TCG_TARGET_HAS_ext8s_i64
--
2.25.1
- [PATCH v3 00/70] TCI fixes and cleanups, Richard Henderson, 2021/02/07
- [PATCH v3 01/70] gdbstub: Fix handle_query_xfer_auxv, Richard Henderson, 2021/02/07
- [PATCH v3 02/70] tcg: Split out tcg_raise_tb_overflow, Richard Henderson, 2021/02/07
- [PATCH v3 04/70] tcg/tci: Merge identical cases in generation, Richard Henderson, 2021/02/07
- [PATCH v3 03/70] tcg: Manage splitwx in tc_ptr_to_region_tree by hand, Richard Henderson, 2021/02/07
- [PATCH v3 05/70] tcg/tci: Remove tci_read_r8,
Richard Henderson <=
- [PATCH v3 06/70] tcg/tci: Remove tci_read_r8s, Richard Henderson, 2021/02/07
- [PATCH v3 07/70] tcg/tci: Remove tci_read_r16, Richard Henderson, 2021/02/07
- [PATCH v3 08/70] tcg/tci: Remove tci_read_r16s, Richard Henderson, 2021/02/07
- [PATCH v3 09/70] tcg/tci: Remove tci_read_r32, Richard Henderson, 2021/02/07
- [PATCH v3 13/70] tcg/tci: Merge extension operations, Richard Henderson, 2021/02/07
- [PATCH v3 14/70] tcg/tci: Remove ifdefs for TCG_TARGET_HAS_ext32[us]_i64, Richard Henderson, 2021/02/07
- [PATCH v3 10/70] tcg/tci: Remove tci_read_r32s, Richard Henderson, 2021/02/07
- [PATCH v3 11/70] tcg/tci: Reduce use of tci_read_r64, Richard Henderson, 2021/02/07
- [PATCH v3 12/70] tcg/tci: Merge basic arithmetic operations, Richard Henderson, 2021/02/07
- [PATCH v3 16/70] tcg/tci: Merge mov, not and neg operations, Richard Henderson, 2021/02/07