[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v8 44/68] target/nios2: Split out helpers for gen_i_cmpxx
From: |
Richard Henderson |
Subject: |
[PATCH v8 44/68] target/nios2: Split out helpers for gen_i_cmpxx |
Date: |
Fri, 22 Apr 2022 09:52:14 -0700 |
Do as little work as possible within the macro.
Split out helper functions and pass in arguments instead.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/nios2/translate.c | 34 +++++++++++++++++++++++++++-------
1 file changed, 27 insertions(+), 7 deletions(-)
diff --git a/target/nios2/translate.c b/target/nios2/translate.c
index a3c63dbbbd..86978ba47a 100644
--- a/target/nios2/translate.c
+++ b/target/nios2/translate.c
@@ -71,6 +71,18 @@ typedef struct {
.a = extract32((code), 27, 5), \
}
+typedef target_ulong ImmFromIType(const InstrIType *);
+
+static target_ulong imm_unsigned(const InstrIType *i)
+{
+ return i->imm16.u;
+}
+
+static target_ulong imm_signed(const InstrIType *i)
+{
+ return i->imm16.s;
+}
+
/* R-Type instruction parsing */
typedef struct {
uint8_t op;
@@ -268,15 +280,23 @@ static void gen_bxx(DisasContext *dc, uint32_t code,
uint32_t flags)
}
/* Comparison instructions */
-#define gen_i_cmpxx(fname, op3) \
-static void (fname)(DisasContext *dc, uint32_t code, uint32_t flags) \
-{ \
- I_TYPE(instr, (code)); \
- tcg_gen_setcondi_tl(flags, cpu_R[instr.b], cpu_R[instr.a], (op3)); \
+static void do_i_cmpxx(DisasContext *dc, uint32_t insn,
+ TCGCond cond, ImmFromIType *imm)
+{
+ I_TYPE(instr, insn);
+
+ if (likely(instr.b != R_ZERO)) {
+ tcg_gen_setcondi_tl(cond, cpu_R[instr.b],
+ load_gpr(dc, instr.a), imm(&instr));
+ }
}
-gen_i_cmpxx(gen_cmpxxsi, instr.imm16.s)
-gen_i_cmpxx(gen_cmpxxui, instr.imm16.u)
+#define gen_i_cmpxx(fname, imm) \
+ static void (fname)(DisasContext *dc, uint32_t code, uint32_t flags) \
+ { do_i_cmpxx(dc, code, flags, imm); }
+
+gen_i_cmpxx(gen_cmpxxsi, imm_signed)
+gen_i_cmpxx(gen_cmpxxui, imm_unsigned)
/* Math/logic instructions */
#define gen_i_math_logic(fname, insn, resimm, op3) \
--
2.34.1
- [PATCH v8 34/68] target/nios2: Handle EXCP_UNALIGN and EXCP_UNALIGND, (continued)
- [PATCH v8 34/68] target/nios2: Handle EXCP_UNALIGN and EXCP_UNALIGND, Richard Henderson, 2022/04/22
- [PATCH v8 35/68] target/nios2: Cleanup set of CR_EXCEPTION for do_interrupt, Richard Henderson, 2022/04/22
- [PATCH v8 36/68] target/nios2: Clean up handling of tlbmisc in do_exception, Richard Henderson, 2022/04/22
- [PATCH v8 38/68] target/nios2: Implement cpuid, Richard Henderson, 2022/04/22
- [PATCH v8 37/68] target/nios2: Prevent writes to read-only or reserved control fields, Richard Henderson, 2022/04/22
- [PATCH v8 39/68] target/nios2: Implement CR_STATUS.RSIE, Richard Henderson, 2022/04/22
- [PATCH v8 40/68] target/nios2: Remove CPU_INTERRUPT_NMI, Richard Henderson, 2022/04/22
- [PATCH v8 41/68] target/nios2: Support division error exception, Richard Henderson, 2022/04/22
- [PATCH v8 42/68] target/nios2: Use tcg_constant_tl, Richard Henderson, 2022/04/22
- [PATCH v8 43/68] target/nios2: Split out named structs for [IRJ]_TYPE, Richard Henderson, 2022/04/22
- [PATCH v8 44/68] target/nios2: Split out helpers for gen_i_cmpxx,
Richard Henderson <=
- [PATCH v8 45/68] target/nios2: Split out helpers for gen_i_math_logic, Richard Henderson, 2022/04/22
- [PATCH v8 46/68] target/nios2: Split out helpers for gen_r_math_logic, Richard Henderson, 2022/04/22
- [PATCH v8 47/68] target/nios2: Split out helpers for gen_rr_mul_high, Richard Henderson, 2022/04/22
- [PATCH v8 48/68] target/nios2: Split out helpers for gen_rr_shift, Richard Henderson, 2022/04/22
- [PATCH v8 49/68] target/nios2: Introduce dest_gpr, Richard Henderson, 2022/04/22