[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [RFC for-4.1 21/25] target/ppc: Style fixes for translate
From: |
Greg Kurz |
Subject: |
Re: [Qemu-ppc] [RFC for-4.1 21/25] target/ppc: Style fixes for translate.c |
Date: |
Mon, 25 Mar 2019 10:56:34 +0100 |
On Fri, 22 Mar 2019 11:15:40 +1100
David Gibson <address@hidden> wrote:
> Signed-off-by: David Gibson <address@hidden>
> ---
> target/ppc/translate.c | 507 +++++++++++++++++++++++++----------------
> 1 file changed, 315 insertions(+), 192 deletions(-)
>
> diff --git a/target/ppc/translate.c b/target/ppc/translate.c
> index 98b37cebc2..67aa128ef1 100644
> --- a/target/ppc/translate.c
> +++ b/target/ppc/translate.c
> @@ -42,8 +42,8 @@
> #define GDBSTUB_SINGLE_STEP 0x4
>
> /* Include definitions for instructions classes and implementations flags */
> -//#define PPC_DEBUG_DISAS
> -//#define DO_PPC_STATISTICS
> +/* #define PPC_DEBUG_DISAS */
> +/* #define DO_PPC_STATISTICS */
>
> #ifdef PPC_DEBUG_DISAS
> # define LOG_DISAS(...) qemu_log_mask(CPU_LOG_TB_IN_ASM, ## __VA_ARGS__)
> @@ -54,9 +54,9 @@
> /* Code translation helpers
> */
>
> /* global register indexes */
> -static char cpu_reg_names[10*3 + 22*4 /* GPR */
> - + 10*4 + 22*5 /* SPE GPRh */
> - + 8*5 /* CRF */];
> +static char cpu_reg_names[10 * 3 + 22 * 4 /* GPR */
> + + 10 * 4 + 22 * 5 /* SPE GPRh */
> + + 8 * 5 /* CRF */];
> static TCGv cpu_gpr[32];
> static TCGv cpu_gprh[32];
> static TCGv_i32 cpu_crf[8];
> @@ -78,7 +78,7 @@ static TCGv_i32 cpu_access_type;
> void ppc_translate_init(void)
> {
> int i;
> - char* p;
> + char *p;
> size_t cpu_reg_names_size;
>
> p = cpu_reg_names;
> @@ -146,7 +146,8 @@ void ppc_translate_init(void)
> offsetof(CPUPPCState, fpscr), "fpscr");
>
> cpu_access_type = tcg_global_mem_new_i32(cpu_env,
> - offsetof(CPUPPCState,
> access_type), "access_type");
> + offsetof(CPUPPCState,
> access_type),
> + "access_type");
> }
>
> /* internal defines */
> @@ -246,8 +247,9 @@ static void gen_exception_err(DisasContext *ctx, uint32_t
> excp, uint32_t error)
> {
> TCGv_i32 t0, t1;
>
> - /* These are all synchronous exceptions, we set the PC back to
> - * the faulting instruction
> + /*
> + * These are all synchronous exceptions, we set the PC back to the
> + * faulting instruction
> */
> if (ctx->exception == POWERPC_EXCP_NONE) {
> gen_update_nip(ctx, ctx->base.pc_next - 4);
> @@ -264,8 +266,9 @@ static void gen_exception(DisasContext *ctx, uint32_t
> excp)
> {
> TCGv_i32 t0;
>
> - /* These are all synchronous exceptions, we set the PC back to
> - * the faulting instruction
> + /*
> + * These are all synchronous exceptions, we set the PC back to the
> + * faulting instruction
> */
> if (ctx->exception == POWERPC_EXCP_NONE) {
> gen_update_nip(ctx, ctx->base.pc_next - 4);
> @@ -320,8 +323,9 @@ static void gen_debug_exception(DisasContext *ctx)
> {
> TCGv_i32 t0;
>
> - /* These are all synchronous exceptions, we set the PC back to
> - * the faulting instruction
> + /*
> + * These are all synchronous exceptions, we set the PC back to the
> + * faulting instruction
> */
> if ((ctx->exception != POWERPC_EXCP_BRANCH) &&
> (ctx->exception != POWERPC_EXCP_SYNC)) {
> @@ -602,9 +606,11 @@ static inline void gen_op_cmp(TCGv arg0, TCGv arg1, int
> s, int crf)
>
> tcg_gen_movi_tl(t0, CRF_EQ);
> tcg_gen_movi_tl(t1, CRF_LT);
> - tcg_gen_movcond_tl((s ? TCG_COND_LT : TCG_COND_LTU), t0, arg0, arg1, t1,
> t0);
> + tcg_gen_movcond_tl((s ? TCG_COND_LT : TCG_COND_LTU),
> + t0, arg0, arg1, t1, t0);
> tcg_gen_movi_tl(t1, CRF_GT);
> - tcg_gen_movcond_tl((s ? TCG_COND_GT : TCG_COND_GTU), t0, arg0, arg1, t1,
> t0);
> + tcg_gen_movcond_tl((s ? TCG_COND_GT : TCG_COND_GTU),
> + t0, arg0, arg1, t1, t0);
>
> tcg_gen_trunc_tl_i32(t, t0);
> tcg_gen_trunc_tl_i32(cpu_crf[crf], cpu_so);
> @@ -840,9 +846,11 @@ static inline void gen_op_arith_add(DisasContext *ctx,
> TCGv ret, TCGv arg1,
>
> if (compute_ca) {
> if (NARROW_MODE(ctx)) {
> - /* Caution: a non-obvious corner case of the spec is that we
> - must produce the *entire* 64-bit addition, but produce the
> - carry into bit 32. */
> + /*
> + * Caution: a non-obvious corner case of the spec is that
> + * we must produce the *entire* 64-bit addition, but
> + * produce the carry into bit 32.
> + */
> TCGv t1 = tcg_temp_new();
> tcg_gen_xor_tl(t1, arg1, arg2); /* add without carry */
> tcg_gen_add_tl(t0, arg1, arg2);
> @@ -1017,12 +1025,13 @@ static inline void gen_op_arith_divw(DisasContext
> *ctx, TCGv ret, TCGv arg1,
> tcg_temp_free_i32(t2);
> tcg_temp_free_i32(t3);
>
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, ret);
> + }
> }
> /* Div functions */
> #define GEN_INT_ARITH_DIVW(name, opc3, sign, compute_ov)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> gen_op_arith_divw(ctx, cpu_gpr[rD(ctx->opcode)],
> \
> cpu_gpr[rA(ctx->opcode)], cpu_gpr[rB(ctx->opcode)],
> \
> @@ -1091,12 +1100,13 @@ static inline void gen_op_arith_divd(DisasContext
> *ctx, TCGv ret, TCGv arg1,
> tcg_temp_free_i64(t2);
> tcg_temp_free_i64(t3);
>
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, ret);
> + }
> }
>
> #define GEN_INT_ARITH_DIVD(name, opc3, sign, compute_ov)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> gen_op_arith_divd(ctx, cpu_gpr[rD(ctx->opcode)],
> \
> cpu_gpr[rA(ctx->opcode)], cpu_gpr[rB(ctx->opcode)],
> \
> @@ -1219,8 +1229,9 @@ static void gen_mulhw(DisasContext *ctx)
> tcg_gen_extu_i32_tl(cpu_gpr[rD(ctx->opcode)], t1);
> tcg_temp_free_i32(t0);
> tcg_temp_free_i32(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mulhwu mulhwu. */
> @@ -1235,8 +1246,9 @@ static void gen_mulhwu(DisasContext *ctx)
> tcg_gen_extu_i32_tl(cpu_gpr[rD(ctx->opcode)], t1);
> tcg_temp_free_i32(t0);
> tcg_temp_free_i32(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mullw mullw. */
> @@ -1255,8 +1267,9 @@ static void gen_mullw(DisasContext *ctx)
> tcg_gen_mul_i32(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> #endif
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mullwo mullwo. */
> @@ -1284,8 +1297,9 @@ static void gen_mullwo(DisasContext *ctx)
>
> tcg_temp_free_i32(t0);
> tcg_temp_free_i32(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mulli */
> @@ -1325,8 +1339,9 @@ static void gen_mulld(DisasContext *ctx)
> {
> tcg_gen_mul_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mulldo mulldo. */
> @@ -1369,9 +1384,11 @@ static inline void gen_op_arith_subf(DisasContext
> *ctx, TCGv ret, TCGv arg1,
> if (compute_ca) {
> /* dest = ~arg1 + arg2 [+ ca]. */
> if (NARROW_MODE(ctx)) {
> - /* Caution: a non-obvious corner case of the spec is that we
> - must produce the *entire* 64-bit addition, but produce the
> - carry into bit 32. */
> + /*
> + * Caution: a non-obvious corner case of the spec is that
> + * we must produce the *entire* 64-bit addition, but
> + * produce the carry into bit 32.
> + */
> TCGv inv1 = tcg_temp_new();
> TCGv t1 = tcg_temp_new();
> tcg_gen_not_tl(inv1, arg1);
> @@ -1404,8 +1421,10 @@ static inline void gen_op_arith_subf(DisasContext
> *ctx, TCGv ret, TCGv arg1,
> gen_op_arith_compute_ca32(ctx, t0, arg1, arg2, cpu_ca32, 1);
> }
> } else if (add_ca) {
> - /* Since we're ignoring carry-out, we can simplify the
> - standard ~arg1 + arg2 + ca to arg2 - arg1 + ca - 1. */
> + /*
> + * Since we're ignoring carry-out, we can simplify the
> + * standard ~arg1 + arg2 + ca to arg2 - arg1 + ca - 1.
> + */
> tcg_gen_sub_tl(t0, arg2, arg1);
> tcg_gen_add_tl(t0, t0, cpu_ca);
> tcg_gen_subi_tl(t0, t0, 1);
> @@ -1493,7 +1512,7 @@ static void gen_nego(DisasContext *ctx)
>
> /*** Integer logical
> ***/
> #define GEN_LOGICAL2(name, tcg_op, opc, type)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> tcg_op(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> \
> cpu_gpr[rB(ctx->opcode)]);
> \
> @@ -1502,7 +1521,7 @@ static void glue(gen_, name)(DisasContext *ctx)
> }
>
> #define GEN_LOGICAL1(name, tcg_op, opc, type)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> tcg_op(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)]);
> \
> if (unlikely(Rc(ctx->opcode) != 0))
> \
> @@ -1517,14 +1536,16 @@ GEN_LOGICAL2(andc, tcg_gen_andc_tl, 0x01,
> PPC_INTEGER);
> /* andi. */
> static void gen_andi_(DisasContext *ctx)
> {
> - tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> UIMM(ctx->opcode));
> + tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> + UIMM(ctx->opcode));
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> }
>
> /* andis. */
> static void gen_andis_(DisasContext *ctx)
> {
> - tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> UIMM(ctx->opcode) << 16);
> + tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> + UIMM(ctx->opcode) << 16);
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> }
>
> @@ -1538,8 +1559,9 @@ static void gen_cntlzw(DisasContext *ctx)
> tcg_gen_extu_i32_tl(cpu_gpr[rA(ctx->opcode)], t);
> tcg_temp_free_i32(t);
>
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* cnttzw */
> @@ -1591,12 +1613,14 @@ static void gen_or(DisasContext *ctx)
> rb = rB(ctx->opcode);
> /* Optimisation for mr. ri case */
> if (rs != ra || rs != rb) {
> - if (rs != rb)
> + if (rs != rb) {
> tcg_gen_or_tl(cpu_gpr[ra], cpu_gpr[rs], cpu_gpr[rb]);
> - else
> + } else {
> tcg_gen_mov_tl(cpu_gpr[ra], cpu_gpr[rs]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + }
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[ra]);
> + }
> } else if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rs]);
> #if defined(TARGET_PPC64)
> @@ -1654,10 +1678,11 @@ static void gen_or(DisasContext *ctx)
> tcg_temp_free(t0);
> }
> #if !defined(CONFIG_USER_ONLY)
> - /* Pause out of TCG otherwise spin loops with smt_low eat too much
> - * CPU and the kernel hangs. This applies to all encodings other
> - * than no-op, e.g., miso(rs=26), yield(27), mdoio(29), mdoom(30),
> - * and all currently undefined.
> + /*
> + * Pause out of TCG otherwise spin loops with smt_low eat too
> + * much CPU and the kernel hangs. This applies to all
> + * encodings other than no-op, e.g., miso(rs=26), yield(27),
> + * mdoio(29), mdoom(30), and all currently undefined.
> */
> gen_pause(ctx);
> #endif
> @@ -1671,12 +1696,15 @@ GEN_LOGICAL2(orc, tcg_gen_orc_tl, 0x0C, PPC_INTEGER);
> static void gen_xor(DisasContext *ctx)
> {
> /* Optimisation for "set to zero" case */
> - if (rS(ctx->opcode) != rB(ctx->opcode))
> - tcg_gen_xor_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> - else
> + if (rS(ctx->opcode) != rB(ctx->opcode)) {
> + tcg_gen_xor_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> + cpu_gpr[rB(ctx->opcode)]);
> + } else {
> tcg_gen_movi_tl(cpu_gpr[rA(ctx->opcode)], 0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + }
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* ori */
> @@ -1699,7 +1727,8 @@ static void gen_oris(DisasContext *ctx)
> /* NOP */
> return;
> }
> - tcg_gen_ori_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)], uimm
> << 16);
> + tcg_gen_ori_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> + uimm << 16);
> }
>
> /* xori */
> @@ -1723,7 +1752,8 @@ static void gen_xoris(DisasContext *ctx)
> /* NOP */
> return;
> }
> - tcg_gen_xori_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)], uimm
> << 16);
> + tcg_gen_xori_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)],
> + uimm << 16);
> }
>
> /* popcntb : PowerPC 2.03 specification */
> @@ -1798,8 +1828,9 @@ GEN_LOGICAL1(extsw, tcg_gen_ext32s_tl, 0x1E, PPC_64B);
> static void gen_cntlzd(DisasContext *ctx)
> {
> tcg_gen_clzi_i64(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)], 64);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* cnttzd */
> @@ -1838,7 +1869,7 @@ static void gen_rlwimi(DisasContext *ctx)
> uint32_t mb = MB(ctx->opcode);
> uint32_t me = ME(ctx->opcode);
>
> - if (sh == (31-me) && mb <= me) {
> + if (sh == (31 - me) && mb <= me) {
> tcg_gen_deposit_tl(t_ra, t_ra, t_rs, sh, me - mb + 1);
> } else {
> target_ulong mask;
> @@ -2141,8 +2172,9 @@ static void gen_slw(DisasContext *ctx)
> tcg_temp_free(t1);
> tcg_temp_free(t0);
> tcg_gen_ext32u_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sraw & sraw. */
> @@ -2150,8 +2182,9 @@ static void gen_sraw(DisasContext *ctx)
> {
> gen_helper_sraw(cpu_gpr[rA(ctx->opcode)], cpu_env,
> cpu_gpr[rS(ctx->opcode)], cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srawi & srawi. */
> @@ -2206,8 +2239,9 @@ static void gen_srw(DisasContext *ctx)
> tcg_gen_shr_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t1);
> tcg_temp_free(t0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> #if defined(TARGET_PPC64)
> @@ -2226,8 +2260,9 @@ static void gen_sld(DisasContext *ctx)
> tcg_gen_shl_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t1);
> tcg_temp_free(t0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srad & srad. */
> @@ -2235,8 +2270,9 @@ static void gen_srad(DisasContext *ctx)
> {
> gen_helper_srad(cpu_gpr[rA(ctx->opcode)], cpu_env,
> cpu_gpr[rS(ctx->opcode)], cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
> /* sradi & sradi. */
> static inline void gen_sradi(DisasContext *ctx, int n)
> @@ -2317,8 +2353,9 @@ static void gen_srd(DisasContext *ctx)
> tcg_gen_shr_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t1);
> tcg_temp_free(t0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
> #endif
>
> @@ -2463,7 +2500,7 @@ GEN_QEMU_STORE_64(st64r, BSWAP_MEMOP(MO_Q))
> #endif
>
> #define GEN_LD(name, ldop, opc, type)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> gen_set_access_type(ctx, ACCESS_INT);
> \
> @@ -2474,7 +2511,7 @@ static void glue(gen_, name)(DisasContext *ctx)
> }
>
> #define GEN_LDU(name, ldop, opc, type)
> \
> -static void glue(gen_, name##u)(DisasContext *ctx)
> \
> +static void glue(gen_, name##u)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> if (unlikely(rA(ctx->opcode) == 0 ||
> \
> @@ -2494,7 +2531,7 @@ static void glue(gen_, name##u)(DisasContext *ctx)
> }
>
> #define GEN_LDUX(name, ldop, opc2, opc3, type)
> \
> -static void glue(gen_, name##ux)(DisasContext *ctx)
> \
> +static void glue(gen_, name##ux)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> if (unlikely(rA(ctx->opcode) == 0 ||
> \
> @@ -2598,8 +2635,9 @@ static void gen_ld(DisasContext *ctx)
> /* ld - ldu */
> gen_qemu_ld64_i64(ctx, cpu_gpr[rD(ctx->opcode)], EA);
> }
> - if (Rc(ctx->opcode))
> + if (Rc(ctx->opcode)) {
> tcg_gen_mov_tl(cpu_gpr[rA(ctx->opcode)], EA);
> + }
> tcg_temp_free(EA);
> }
>
> @@ -2669,7 +2707,7 @@ static void gen_lq(DisasContext *ctx)
>
> /*** Integer store
> ***/
> #define GEN_ST(name, stop, opc, type)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> gen_set_access_type(ctx, ACCESS_INT);
> \
> @@ -2680,7 +2718,7 @@ static void glue(gen_, name)(DisasContext *ctx)
> }
>
> #define GEN_STU(name, stop, opc, type)
> \
> -static void glue(gen_, stop##u)(DisasContext *ctx)
> \
> +static void glue(gen_, stop##u)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> if (unlikely(rA(ctx->opcode) == 0)) {
> \
> @@ -2699,7 +2737,7 @@ static void glue(gen_, stop##u)(DisasContext *ctx)
> }
>
> #define GEN_STUX(name, stop, opc2, opc3, type)
> \
> -static void glue(gen_, name##ux)(DisasContext *ctx)
> \
> +static void glue(gen_, name##ux)(DisasContext *ctx)
> \
> {
> \
> TCGv EA;
> \
> if (unlikely(rA(ctx->opcode) == 0)) {
> \
> @@ -2847,8 +2885,9 @@ static void gen_std(DisasContext *ctx)
> EA = tcg_temp_new();
> gen_addr_imm_index(ctx, EA, 0x03);
> gen_qemu_st64_i64(ctx, cpu_gpr[rs], EA);
> - if (Rc(ctx->opcode))
> + if (Rc(ctx->opcode)) {
> tcg_gen_mov_tl(cpu_gpr[rA(ctx->opcode)], EA);
> + }
> tcg_temp_free(EA);
> }
> }
> @@ -2916,10 +2955,11 @@ static void gen_stmw(DisasContext *ctx)
> /*** Integer load and store strings
> ***/
>
> /* lswi */
> -/* PowerPC32 specification says we must generate an exception if
> - * rA is in the range of registers to be loaded.
> - * In an other hand, IBM says this is valid, but rA won't be loaded.
> - * For now, I'll follow the spec...
> +/*
> + * PowerPC32 specification says we must generate an exception if rA is
> + * in the range of registers to be loaded. In an other hand, IBM says
> + * this is valid, but rA won't be loaded. For now, I'll follow the
> + * spec...
> */
Merge with the preceding comment ?
> static void gen_lswi(DisasContext *ctx)
> {
> @@ -2934,8 +2974,9 @@ static void gen_lswi(DisasContext *ctx)
> gen_align_no_le(ctx);
> return;
> }
> - if (nb == 0)
> + if (nb == 0) {
> nb = 32;
> + }
> nr = DIV_ROUND_UP(nb, 4);
> if (unlikely(lsw_reg_in_range(start, nr, ra))) {
> gen_inval_exception(ctx, POWERPC_EXCP_INVAL_LSWX);
> @@ -2989,8 +3030,9 @@ static void gen_stswi(DisasContext *ctx)
> gen_set_access_type(ctx, ACCESS_INT);
> t0 = tcg_temp_new();
> gen_addr_register(ctx, t0);
> - if (nb == 0)
> + if (nb == 0) {
> nb = 32;
> + }
> t1 = tcg_const_i32(nb);
> t2 = tcg_const_i32(rS(ctx->opcode));
> gen_helper_stsw(cpu_env, t0, t1, t2);
> @@ -3363,8 +3405,10 @@ static void gen_conditional_store(DisasContext *ctx,
> TCGMemOp memop)
>
> gen_set_label(l1);
>
> - /* Address mismatch implies failure. But we still need to provide the
> - memory barrier semantics of the instruction. */
> + /*
> + * Address mismatch implies failure. But we still need to provide
> + * the memory barrier semantics of the instruction.
> + */
> tcg_gen_mb(TCG_MO_ALL | TCG_BAR_STRL);
> tcg_gen_trunc_tl_i32(cpu_crf[0], cpu_so);
>
> @@ -3639,8 +3683,9 @@ static void gen_rvwinkle(DisasContext *ctx)
> static inline void gen_update_cfar(DisasContext *ctx, target_ulong nip)
> {
> #if defined(TARGET_PPC64)
> - if (ctx->has_cfar)
> + if (ctx->has_cfar) {
> tcg_gen_movi_tl(cpu_cfar, nip);
> + }
> #endif
> }
>
> @@ -3732,17 +3777,19 @@ static void gen_bcond(DisasContext *ctx, int type)
>
> if (type == BCOND_LR || type == BCOND_CTR || type == BCOND_TAR) {
> target = tcg_temp_local_new();
> - if (type == BCOND_CTR)
> + if (type == BCOND_CTR) {
> tcg_gen_mov_tl(target, cpu_ctr);
> - else if (type == BCOND_TAR)
> + } else if (type == BCOND_TAR) {
> gen_load_spr(target, SPR_TAR);
> - else
> + } else {
> tcg_gen_mov_tl(target, cpu_lr);
> + }
> } else {
> target = NULL;
> }
> - if (LK(ctx->opcode))
> + if (LK(ctx->opcode)) {
> gen_setlr(ctx, ctx->base.pc_next);
> + }
> l1 = gen_new_label();
> if ((bo & 0x4) == 0) {
> /* Decrement and test CTR */
> @@ -3825,7 +3872,7 @@ static void gen_bctar(DisasContext *ctx)
>
> /*** Condition register logical
> ***/
> #define GEN_CRLOGIC(name, tcg_op, opc)
> \
> -static void glue(gen_, name)(DisasContext *ctx)
> \
> +static void glue(gen_, name)(DisasContext *ctx)
> \
> {
> \
> uint8_t bitmask;
> \
> int sh;
> \
> @@ -3886,7 +3933,8 @@ static void gen_rfi(DisasContext *ctx)
> #if defined(CONFIG_USER_ONLY)
> GEN_PRIV;
> #else
> - /* This instruction doesn't exist anymore on 64-bit server
> + /*
> + * This instruction doesn't exist anymore on 64-bit server
> * processors compliant with arch 2.x
> */
> if (ctx->insns_flags & PPC_SEGMENT_64B) {
> @@ -4125,7 +4173,7 @@ static void gen_mfcr(DisasContext *ctx)
> if (likely(ctx->opcode & 0x00100000)) {
> crm = CRM(ctx->opcode);
> if (likely(crm && ((crm & (crm - 1)) == 0))) {
> - crn = ctz32 (crm);
> + crn = ctz32(crm);
> tcg_gen_extu_i32_tl(cpu_gpr[rD(ctx->opcode)], cpu_crf[7 - crn]);
> tcg_gen_shli_tl(cpu_gpr[rD(ctx->opcode)],
> cpu_gpr[rD(ctx->opcode)], crn * 4);
> @@ -4190,7 +4238,8 @@ static inline void gen_op_mfspr(DisasContext *ctx)
> (*read_cb)(ctx, rD(ctx->opcode), sprn);
> } else {
> /* Privilege exception */
> - /* This is a hack to avoid warnings when running Linux:
> + /*
> + * This is a hack to avoid warnings when running Linux:
> * this OS breaks the PowerPC virtualisation model,
> * allowing userland application to read the PVR
> */
Merge with the preceding comment ?
> @@ -4213,8 +4262,9 @@ static inline void gen_op_mfspr(DisasContext *ctx)
> "Trying to read invalid spr %d (0x%03x) at "
> TARGET_FMT_lx "\n", sprn, sprn, ctx->base.pc_next - 4);
>
> - /* The behaviour depends on MSR:PR and SPR# bit 0x10,
> - * it can generate a priv, a hv emu or a no-op
> + /*
> + * The behaviour depends on MSR:PR and SPR# bit 0x10, it can
> + * generate a priv, a hv emu or a no-op
> */
> if (sprn & 0x10) {
> if (ctx->pr) {
> @@ -4248,7 +4298,7 @@ static void gen_mtcrf(DisasContext *ctx)
> if (likely((ctx->opcode & 0x00100000))) {
> if (crm && ((crm & (crm - 1)) == 0)) {
> TCGv_i32 temp = tcg_temp_new_i32();
> - crn = ctz32 (crm);
> + crn = ctz32(crm);
> tcg_gen_trunc_tl_i32(temp, cpu_gpr[rS(ctx->opcode)]);
> tcg_gen_shri_i32(temp, temp, crn * 4);
> tcg_gen_andi_i32(cpu_crf[7 - crn], temp, 0xf);
> @@ -4277,14 +4327,17 @@ static void gen_mtmsrd(DisasContext *ctx)
> if (ctx->opcode & 0x00010000) {
> /* Special form that does not need any synchronisation */
> TCGv t0 = tcg_temp_new();
> - tcg_gen_andi_tl(t0, cpu_gpr[rS(ctx->opcode)], (1 << MSR_RI) | (1 <<
> MSR_EE));
> - tcg_gen_andi_tl(cpu_msr, cpu_msr, ~(target_ulong)((1 << MSR_RI) | (1
> << MSR_EE)));
> + tcg_gen_andi_tl(t0, cpu_gpr[rS(ctx->opcode)],
> + (1 << MSR_RI) | (1 << MSR_EE));
> + tcg_gen_andi_tl(cpu_msr, cpu_msr,
> + ~(target_ulong)((1 << MSR_RI) | (1 << MSR_EE)));
> tcg_gen_or_tl(cpu_msr, cpu_msr, t0);
> tcg_temp_free(t0);
> } else {
> - /* XXX: we need to update nip before the store
> - * if we enter power saving mode, we will exit the loop
> - * directly from ppc_store_msr
> + /*
> + * XXX: we need to update nip before the store if we enter
> + * power saving mode, we will exit the loop directly from
> + * ppc_store_msr
> */
> if (tb_cflags(ctx->base.tb) & CF_USE_ICOUNT) {
> gen_io_start();
> @@ -4310,16 +4363,19 @@ static void gen_mtmsr(DisasContext *ctx)
> if (ctx->opcode & 0x00010000) {
> /* Special form that does not need any synchronisation */
> TCGv t0 = tcg_temp_new();
> - tcg_gen_andi_tl(t0, cpu_gpr[rS(ctx->opcode)], (1 << MSR_RI) | (1 <<
> MSR_EE));
> - tcg_gen_andi_tl(cpu_msr, cpu_msr, ~(target_ulong)((1 << MSR_RI) | (1
> << MSR_EE)));
> + tcg_gen_andi_tl(t0, cpu_gpr[rS(ctx->opcode)],
> + (1 << MSR_RI) | (1 << MSR_EE));
> + tcg_gen_andi_tl(cpu_msr, cpu_msr,
> + ~(target_ulong)((1 << MSR_RI) | (1 << MSR_EE)));
> tcg_gen_or_tl(cpu_msr, cpu_msr, t0);
> tcg_temp_free(t0);
> } else {
> TCGv msr = tcg_temp_new();
>
> - /* XXX: we need to update nip before the store
> - * if we enter power saving mode, we will exit the loop
> - * directly from ppc_store_msr
> + /*
> + * XXX: we need to update nip before the store if we enter
> + * power saving mode, we will exit the loop directly from
> + * ppc_store_msr
> */
> if (tb_cflags(ctx->base.tb) & CF_USE_ICOUNT) {
> gen_io_start();
> @@ -4383,8 +4439,9 @@ static void gen_mtspr(DisasContext *ctx)
> TARGET_FMT_lx "\n", sprn, sprn, ctx->base.pc_next - 4);
>
>
> - /* The behaviour depends on MSR:PR and SPR# bit 0x10,
> - * it can generate a priv, a hv emu or a no-op
> + /*
> + * The behaviour depends on MSR:PR and SPR# bit 0x10, it can
> + * generate a priv, a hv emu or a no-op
> */
> if (sprn & 0x10) {
> if (ctx->pr) {
> @@ -4494,36 +4551,40 @@ static void gen_dcbstep(DisasContext *ctx)
> /* dcbt */
> static void gen_dcbt(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> /* dcbtep */
> static void gen_dcbtep(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> /* dcbtst */
> static void gen_dcbtst(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> /* dcbtstep */
> static void gen_dcbtstep(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> @@ -4621,8 +4682,9 @@ static void gen_icbiep(DisasContext *ctx)
> /* dcba */
> static void gen_dcba(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a store by the MMU
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a store by the MMU
> * but does not generate any exception
> */
> }
> @@ -4989,8 +5051,9 @@ static void gen_abs(DisasContext *ctx)
> gen_set_label(l1);
> tcg_gen_mov_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> gen_set_label(l2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* abso - abso. */
> @@ -5012,8 +5075,9 @@ static void gen_abso(DisasContext *ctx)
> gen_set_label(l2);
> tcg_gen_mov_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> gen_set_label(l3);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* clcs */
> @@ -5030,8 +5094,9 @@ static void gen_div(DisasContext *ctx)
> {
> gen_helper_div(cpu_gpr[rD(ctx->opcode)], cpu_env,
> cpu_gpr[rA(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* divo - divo. */
> @@ -5039,8 +5104,9 @@ static void gen_divo(DisasContext *ctx)
> {
> gen_helper_divo(cpu_gpr[rD(ctx->opcode)], cpu_env,
> cpu_gpr[rA(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* divs - divs. */
> @@ -5048,8 +5114,9 @@ static void gen_divs(DisasContext *ctx)
> {
> gen_helper_divs(cpu_gpr[rD(ctx->opcode)], cpu_env,
> cpu_gpr[rA(ctx->opcode)],
> cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* divso - divso. */
> @@ -5057,8 +5124,9 @@ static void gen_divso(DisasContext *ctx)
> {
> gen_helper_divso(cpu_gpr[rD(ctx->opcode)], cpu_env,
> cpu_gpr[rA(ctx->opcode)], cpu_gpr[rB(ctx->opcode)]);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* doz - doz. */
> @@ -5066,14 +5134,17 @@ static void gen_doz(DisasContext *ctx)
> {
> TCGLabel *l1 = gen_new_label();
> TCGLabel *l2 = gen_new_label();
> - tcg_gen_brcond_tl(TCG_COND_GE, cpu_gpr[rB(ctx->opcode)],
> cpu_gpr[rA(ctx->opcode)], l1);
> - tcg_gen_sub_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rB(ctx->opcode)],
> cpu_gpr[rA(ctx->opcode)]);
> + tcg_gen_brcond_tl(TCG_COND_GE, cpu_gpr[rB(ctx->opcode)],
> + cpu_gpr[rA(ctx->opcode)], l1);
> + tcg_gen_sub_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rB(ctx->opcode)],
> + cpu_gpr[rA(ctx->opcode)]);
> tcg_gen_br(l2);
> gen_set_label(l1);
> tcg_gen_movi_tl(cpu_gpr[rD(ctx->opcode)], 0);
> gen_set_label(l2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* dozo - dozo. */
> @@ -5086,7 +5157,8 @@ static void gen_dozo(DisasContext *ctx)
> TCGv t2 = tcg_temp_new();
> /* Start with XER OV disabled, the most likely case */
> tcg_gen_movi_tl(cpu_ov, 0);
> - tcg_gen_brcond_tl(TCG_COND_GE, cpu_gpr[rB(ctx->opcode)],
> cpu_gpr[rA(ctx->opcode)], l1);
> + tcg_gen_brcond_tl(TCG_COND_GE, cpu_gpr[rB(ctx->opcode)],
> + cpu_gpr[rA(ctx->opcode)], l1);
> tcg_gen_sub_tl(t0, cpu_gpr[rB(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> tcg_gen_xor_tl(t1, cpu_gpr[rB(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> tcg_gen_xor_tl(t2, cpu_gpr[rA(ctx->opcode)], t0);
> @@ -5102,8 +5174,9 @@ static void gen_dozo(DisasContext *ctx)
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* dozi */
> @@ -5118,8 +5191,9 @@ static void gen_dozi(DisasContext *ctx)
> gen_set_label(l1);
> tcg_gen_movi_tl(cpu_gpr[rD(ctx->opcode)], 0);
> gen_set_label(l2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* lscbx - lscbx. */
> @@ -5137,8 +5211,9 @@ static void gen_lscbx(DisasContext *ctx)
> tcg_temp_free_i32(t3);
> tcg_gen_andi_tl(cpu_xer, cpu_xer, ~0x7F);
> tcg_gen_or_tl(cpu_xer, cpu_xer, t0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, t0);
> + }
> tcg_temp_free(t0);
> }
>
> @@ -5164,8 +5239,9 @@ static void gen_maskg(DisasContext *ctx)
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> tcg_temp_free(t3);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* maskir - maskir. */
> @@ -5178,8 +5254,9 @@ static void gen_maskir(DisasContext *ctx)
> tcg_gen_or_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* mul - mul. */
> @@ -5198,8 +5275,9 @@ static void gen_mul(DisasContext *ctx)
> tcg_temp_free_i64(t0);
> tcg_temp_free_i64(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* mulo - mulo. */
> @@ -5226,8 +5304,9 @@ static void gen_mulo(DisasContext *ctx)
> tcg_temp_free_i64(t0);
> tcg_temp_free_i64(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* nabs - nabs. */
> @@ -5241,8 +5320,9 @@ static void gen_nabs(DisasContext *ctx)
> gen_set_label(l1);
> tcg_gen_neg_tl(cpu_gpr[rD(ctx->opcode)], cpu_gpr[rA(ctx->opcode)]);
> gen_set_label(l2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* nabso - nabso. */
> @@ -5258,8 +5338,9 @@ static void gen_nabso(DisasContext *ctx)
> gen_set_label(l2);
> /* nabs never overflows */
> tcg_gen_movi_tl(cpu_ov, 0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rD(ctx->opcode)]);
> + }
> }
>
> /* rlmi - rlmi. */
> @@ -5271,11 +5352,13 @@ static void gen_rlmi(DisasContext *ctx)
> tcg_gen_andi_tl(t0, cpu_gpr[rB(ctx->opcode)], 0x1F);
> tcg_gen_rotl_tl(t0, cpu_gpr[rS(ctx->opcode)], t0);
> tcg_gen_andi_tl(t0, t0, MASK(mb, me));
> - tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rA(ctx->opcode)],
> ~MASK(mb, me));
> + tcg_gen_andi_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rA(ctx->opcode)],
> + ~MASK(mb, me));
> tcg_gen_or_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rA(ctx->opcode)], t0);
> tcg_temp_free(t0);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* rrib - rrib. */
> @@ -5292,8 +5375,9 @@ static void gen_rrib(DisasContext *ctx)
> tcg_gen_or_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sle - sle. */
> @@ -5310,8 +5394,9 @@ static void gen_sle(DisasContext *ctx)
> gen_store_spr(SPR_MQ, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sleq - sleq. */
> @@ -5332,8 +5417,9 @@ static void gen_sleq(DisasContext *ctx)
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sliq - sliq. */
> @@ -5349,8 +5435,9 @@ static void gen_sliq(DisasContext *ctx)
> gen_store_spr(SPR_MQ, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* slliq - slliq. */
> @@ -5367,8 +5454,9 @@ static void gen_slliq(DisasContext *ctx)
> tcg_gen_or_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sllq - sllq. */
> @@ -5396,8 +5484,9 @@ static void gen_sllq(DisasContext *ctx)
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* slq - slq. */
> @@ -5419,8 +5508,9 @@ static void gen_slq(DisasContext *ctx)
> gen_set_label(l1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sraiq - sraiq. */
> @@ -5442,8 +5532,9 @@ static void gen_sraiq(DisasContext *ctx)
> tcg_gen_sari_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)], sh);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sraq - sraq. */
> @@ -5475,8 +5566,9 @@ static void gen_sraq(DisasContext *ctx)
> gen_set_label(l2);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sre - sre. */
> @@ -5493,8 +5585,9 @@ static void gen_sre(DisasContext *ctx)
> gen_store_spr(SPR_MQ, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srea - srea. */
> @@ -5508,8 +5601,9 @@ static void gen_srea(DisasContext *ctx)
> tcg_gen_sar_tl(cpu_gpr[rA(ctx->opcode)], cpu_gpr[rS(ctx->opcode)], t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sreq */
> @@ -5530,8 +5624,9 @@ static void gen_sreq(DisasContext *ctx)
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* sriq */
> @@ -5547,8 +5642,9 @@ static void gen_sriq(DisasContext *ctx)
> gen_store_spr(SPR_MQ, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srliq */
> @@ -5565,8 +5661,9 @@ static void gen_srliq(DisasContext *ctx)
> tcg_gen_or_tl(cpu_gpr[rA(ctx->opcode)], t0, t1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srlq */
> @@ -5595,8 +5692,9 @@ static void gen_srlq(DisasContext *ctx)
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> tcg_temp_free(t2);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* srq */
> @@ -5618,8 +5716,9 @@ static void gen_srq(DisasContext *ctx)
> gen_set_label(l1);
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> - if (unlikely(Rc(ctx->opcode) != 0))
> + if (unlikely(Rc(ctx->opcode) != 0)) {
> gen_set_Rc0(ctx, cpu_gpr[rA(ctx->opcode)]);
> + }
> }
>
> /* PowerPC 602 specific instructions */
> @@ -5737,8 +5836,9 @@ static void gen_mfsri(DisasContext *ctx)
> tcg_gen_extract_tl(t0, t0, 28, 4);
> gen_helper_load_sr(cpu_gpr[rd], cpu_env, t0);
> tcg_temp_free(t0);
> - if (ra != 0 && ra != rd)
> + if (ra != 0 && ra != rd) {
> tcg_gen_mov_tl(cpu_gpr[ra], cpu_gpr[rd]);
> + }
> #endif /* defined(CONFIG_USER_ONLY) */
> }
>
> @@ -6115,9 +6215,10 @@ static void gen_dcread(DisasContext *ctx)
> /* icbt */
> static void gen_icbt_40x(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> @@ -6408,7 +6509,7 @@ static void gen_tlbilx_booke206(DisasContext *ctx)
> t0 = tcg_temp_new();
> gen_addr_reg_index(ctx, t0);
>
> - switch((ctx->opcode >> 21) & 0x3) {
> + switch ((ctx->opcode >> 21) & 0x3) {
> case 0:
> gen_helper_booke206_tlbilx0(cpu_env, t0);
> break;
> @@ -6442,8 +6543,9 @@ static void gen_wrtee(DisasContext *ctx)
> tcg_gen_andi_tl(cpu_msr, cpu_msr, ~(1 << MSR_EE));
> tcg_gen_or_tl(cpu_msr, cpu_msr, t0);
> tcg_temp_free(t0);
> - /* Stop translation to have a chance to raise an exception
> - * if we just set msr_ee to 1
> + /*
> + * Stop translation to have a chance to raise an exception if we
> + * just set msr_ee to 1
> */
> gen_stop_exception(ctx);
> #endif /* defined(CONFIG_USER_ONLY) */
> @@ -6497,9 +6599,10 @@ static void gen_msync_4xx(DisasContext *ctx)
> /* icbt */
> static void gen_icbt_440(DisasContext *ctx)
> {
> - /* interpreted as no-op */
> - /* XXX: specification say this is treated as a load by the MMU
> - * but does not generate any exception
> + /*
> + * interpreted as no-op
> + * XXX: specification say this is treated as a load by the MMU but
> + * does not generate any exception
> */
> }
>
> @@ -6595,7 +6698,8 @@ static inline void gen_##name(DisasContext *ctx)
> \
> gen_exception_err(ctx, POWERPC_EXCP_FU, FSCR_IC_TM); \
> return; \
> } \
> - /* Because tbegin always fails in QEMU, these user \
> + /* \
> + * Because tbegin always fails in QEMU, these user \
> * space instructions all have a simple implementation: \
> * \
> * CR[0] = 0b0 || MSR[TS] || 0b0 \
> @@ -6611,17 +6715,18 @@ GEN_TM_NOOP(tabortwci);
> GEN_TM_NOOP(tabortdc);
> GEN_TM_NOOP(tabortdci);
> GEN_TM_NOOP(tsr);
> +
> static inline void gen_cp_abort(DisasContext *ctx)
> {
> - // Do Nothing
> + /* Do Nothing */
> }
>
> #define GEN_CP_PASTE_NOOP(name) \
> static inline void gen_##name(DisasContext *ctx) \
> { \
> - /* Generate invalid exception until \
> - * we have an implementation of the copy \
> - * paste facility \
> + /* \
> + * Generate invalid exception until we have an \
> + * implementation of the copy paste facility \
> */ \
> gen_invalid(ctx); \
> }
> @@ -6635,8 +6740,9 @@ static void gen_tcheck(DisasContext *ctx)
> gen_exception_err(ctx, POWERPC_EXCP_FU, FSCR_IC_TM);
> return;
> }
> - /* Because tbegin always fails, the tcheck implementation
> - * is simple:
> + /*
> + * Because tbegin always fails, the tcheck implementation is
> + * simple:
> *
> * CR[CRF] = TDOOMED || MSR[TS] || 0b0
> * = 0b1 || 0b00 || 0b0
> @@ -6648,7 +6754,7 @@ static void gen_tcheck(DisasContext *ctx)
> #define GEN_TM_PRIV_NOOP(name) \
> static inline void gen_##name(DisasContext *ctx) \
> { \
> - gen_priv_exception(ctx, POWERPC_EXCP_PRIV_OPC); \
> + gen_priv_exception(ctx, POWERPC_EXCP_PRIV_OPC); \
> }
>
> #else
> @@ -6661,7 +6767,8 @@ static inline void gen_##name(DisasContext *ctx)
> \
> gen_exception_err(ctx, POWERPC_EXCP_FU, FSCR_IC_TM); \
> return; \
> } \
> - /* Because tbegin always fails, the implementation is \
> + /* \
> + * Because tbegin always fails, the implementation is \
> * simple: \
> * \
> * CR[0] = 0b0 || MSR[TS] || 0b0 \
> @@ -6943,8 +7050,10 @@ GEN_HANDLER2(slbmfev, "slbmfev", 0x1F, 0x13, 0x1A,
> 0x001F0001, PPC_SEGMENT_64B),
> GEN_HANDLER2(slbfee_, "slbfee.", 0x1F, 0x13, 0x1E, 0x001F0000,
> PPC_SEGMENT_64B),
> #endif
> GEN_HANDLER(tlbia, 0x1F, 0x12, 0x0B, 0x03FFFC01, PPC_MEM_TLBIA),
> -/* XXX Those instructions will need to be handled differently for
> - * different ISA versions */
> +/*
> + * XXX Those instructions will need to be handled differently for
> + * different ISA versions
> + */
> GEN_HANDLER(tlbiel, 0x1F, 0x12, 0x08, 0x001F0001, PPC_MEM_TLBIE),
> GEN_HANDLER(tlbie, 0x1F, 0x12, 0x09, 0x001F0001, PPC_MEM_TLBIE),
> GEN_HANDLER_E(tlbiel, 0x1F, 0x12, 0x08, 0x00100001, PPC_NONE, PPC2_ISA300),
> @@ -7415,24 +7524,28 @@ void ppc_cpu_dump_state(CPUState *cs, FILE *f,
> fprintf_function cpu_fprintf,
> );
> #endif
> for (i = 0; i < 32; i++) {
> - if ((i & (RGPL - 1)) == 0)
> + if ((i & (RGPL - 1)) == 0) {
> cpu_fprintf(f, "GPR%02d", i);
> + }
> cpu_fprintf(f, " %016" PRIx64, ppc_dump_gpr(env, i));
> - if ((i & (RGPL - 1)) == (RGPL - 1))
> + if ((i & (RGPL - 1)) == (RGPL - 1)) {
> cpu_fprintf(f, "\n");
> + }
> }
> cpu_fprintf(f, "CR ");
> - for (i = 0; i < 8; i++)
> + for (i = 0; i < 8; i++) {
> cpu_fprintf(f, "%01x", env->crf[i]);
> + }
> cpu_fprintf(f, " [");
> for (i = 0; i < 8; i++) {
> char a = '-';
> - if (env->crf[i] & 0x08)
> + if (env->crf[i] & 0x08) {
> a = 'L';
> - else if (env->crf[i] & 0x04)
> + } else if (env->crf[i] & 0x04) {
> a = 'G';
> - else if (env->crf[i] & 0x02)
> + } else if (env->crf[i] & 0x02) {
> a = 'E';
> + }
> cpu_fprintf(f, " %c%c", a, env->crf[i] & 0x01 ? 'O' : ' ');
> }
> cpu_fprintf(f, " ] RES " TARGET_FMT_lx "\n",
> @@ -7514,8 +7627,9 @@ void ppc_cpu_dump_state(CPUState *cs, FILE *f,
> fprintf_function cpu_fprintf,
> }
> #endif
>
> - if (env->spr_cb[SPR_LPCR].name)
> + if (env->spr_cb[SPR_LPCR].name) {
> cpu_fprintf(f, " LPCR " TARGET_FMT_lx "\n", env->spr[SPR_LPCR]);
> + }
>
> switch (env->mmu_model) {
> case POWERPC_MMU_32B:
> @@ -7582,8 +7696,9 @@ void ppc_cpu_dump_statistics(CPUState *cs, FILE*f,
> t3 = ind_table(handler);
> for (op3 = 0; op3 < 32; op3++) {
> handler = t3[op3];
> - if (handler->count == 0)
> + if (handler->count == 0) {
> continue;
> + }
> cpu_fprintf(f, "%02x %02x %02x (%02x %04d) %16s: "
> "%016" PRIx64 " %" PRId64 "\n",
> op1, op2, op3, op1, (op3 << 5) | op2,
> @@ -7591,8 +7706,9 @@ void ppc_cpu_dump_statistics(CPUState *cs, FILE*f,
> handler->count, handler->count);
> }
> } else {
> - if (handler->count == 0)
> + if (handler->count == 0) {
> continue;
> + }
> cpu_fprintf(f, "%02x %02x (%02x %04d) %16s: "
> "%016" PRIx64 " %" PRId64 "\n",
> op1, op2, op1, op2, handler->oname,
> @@ -7600,8 +7716,9 @@ void ppc_cpu_dump_statistics(CPUState *cs, FILE*f,
> }
> }
> } else {
> - if (handler->count == 0)
> + if (handler->count == 0) {
> continue;
> + }
> cpu_fprintf(f, "%02x (%02x ) %16s: %016" PRIx64
> " %" PRId64 "\n",
> op1, op1, handler->oname,
> @@ -7641,14 +7758,16 @@ static void
> ppc_tr_init_disas_context(DisasContextBase *dcbase, CPUState *cs)
> || (env->mmu_model & POWERPC_MMU_64B);
>
> ctx->fpu_enabled = !!msr_fp;
> - if ((env->flags & POWERPC_FLAG_SPE) && msr_spe)
> + if ((env->flags & POWERPC_FLAG_SPE) && msr_spe) {
> ctx->spe_enabled = !!msr_spe;
> - else
> + } else {
> ctx->spe_enabled = false;
> - if ((env->flags & POWERPC_FLAG_VRE) && msr_vr)
> + }
> + if ((env->flags & POWERPC_FLAG_VRE) && msr_vr) {
> ctx->altivec_enabled = !!msr_vr;
> - else
> + } else {
> ctx->altivec_enabled = false;
> + }
> if ((env->flags & POWERPC_FLAG_VSX) && msr_vsx) {
> ctx->vsx_enabled = !!msr_vsx;
> } else {
> @@ -7662,12 +7781,14 @@ static void
> ppc_tr_init_disas_context(DisasContextBase *dcbase, CPUState *cs)
> }
> #endif
> ctx->gtse = !!(env->spr[SPR_LPCR] & LPCR_GTSE);
> - if ((env->flags & POWERPC_FLAG_SE) && msr_se)
> + if ((env->flags & POWERPC_FLAG_SE) && msr_se) {
> ctx->singlestep_enabled = CPU_SINGLE_STEP;
> - else
> + } else {
> ctx->singlestep_enabled = 0;
> - if ((env->flags & POWERPC_FLAG_BE) && msr_be)
> + }
> + if ((env->flags & POWERPC_FLAG_BE) && msr_be) {
> ctx->singlestep_enabled |= CPU_BRANCH_STEP;
> + }
> if ((env->flags & POWERPC_FLAG_DE) && msr_de) {
> ctx->singlestep_enabled = 0;
> target_ulong dbcr0 = env->spr[SPR_BOOKE_DBCR0];
> @@ -7682,7 +7803,7 @@ static void ppc_tr_init_disas_context(DisasContextBase
> *dcbase, CPUState *cs)
> if (unlikely(ctx->base.singlestep_enabled)) {
> ctx->singlestep_enabled |= GDBSTUB_SINGLE_STEP;
> }
> -#if defined (DO_SINGLE_STEP) && 0
> +#if defined(DO_SINGLE_STEP) && 0
> /* Single step trace mode */
> msr_se = 1;
> #endif
> @@ -7707,10 +7828,12 @@ static bool ppc_tr_breakpoint_check(DisasContextBase
> *dcbase, CPUState *cs,
>
> gen_debug_exception(ctx);
> dcbase->is_jmp = DISAS_NORETURN;
> - /* The address covered by the breakpoint must be included in
> - [tb->pc, tb->pc + tb->size) in order to for it to be
> - properly cleared -- thus we increment the PC here so that
> - the logic setting tb->size below does the right thing. */
> + /*
> + * The address covered by the breakpoint must be included in
> + * [tb->pc, tb->pc + tb->size) in order to for it to be properly
> + * cleared -- thus we increment the PC here so that the logic
> + * setting tb->size below does the right thing.
> + */
> ctx->base.pc_next += 4;
> return true;
> }
- [Qemu-ppc] [RFC for-4.1 01/25] spapr: Support NVIDIA V100 GPU with NVLink2, (continued)
- [Qemu-ppc] [RFC for-4.1 01/25] spapr: Support NVIDIA V100 GPU with NVLink2, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 12/25] target/ppc: Style fixes for machine.c, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 10/25] target/ppc: Style fixes for helper_regs.h, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 02/25] hw/ppc/prep: Drop useless inclusion of "hw/input/i8042.h", David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 04/25] target/ppc: Style fixes for cpu.[ch], David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 21/25] target/ppc: Style fixes for translate.c, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 07/25] target/ppc: Style fixes for dfp_helper.c, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 18/25] target/ppc: Style fixes for mmu_helper.c, David Gibson, 2019/03/21
- [Qemu-ppc] [RFC for-4.1 20/25] target/ppc: Style fixes for translate_init.inc.c, David Gibson, 2019/03/21