From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: Aleksandar Markovic <aleksandar.markovic@rt-rk.com>,
qemu-devel@nongnu.org
Cc: arikalo@wavecomp.com, amarkovic@wavecomp.com
Subject: Re: [Qemu-devel] [PATCH 4/4] target/mips: Fix if-else arms checkpatch errors in translate.c
Date: Thu, 20 Jun 2019 13:11:53 +0200 [thread overview]
Message-ID: <a1ac2dbf-9c7e-24a2-d17b-99cbe08d6640@redhat.com> (raw)
In-Reply-To: <1561022396-20649-5-git-send-email-aleksandar.markovic@rt-rk.com>
On 6/20/19 11:19 AM, Aleksandar Markovic wrote:
> From: Aleksandar Markovic <amarkovic@wavecomp.com>
>
> Remove if-else-arms-related checkpatch errors.
>
> Signed-off-by: Aleksandar Markovic <amarkovic@wavecomp.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> target/mips/translate.c | 106 ++++++++++++++++++++++++++++++------------------
> 1 file changed, 66 insertions(+), 40 deletions(-)
>
> diff --git a/target/mips/translate.c b/target/mips/translate.c
> index 324b32a..3558b2e 100644
> --- a/target/mips/translate.c
> +++ b/target/mips/translate.c
> @@ -2619,16 +2619,18 @@ static const char * const mxuregnames[] = {
> /* General purpose registers moves. */
> static inline void gen_load_gpr(TCGv t, int reg)
> {
> - if (reg == 0)
> + if (reg == 0) {
> tcg_gen_movi_tl(t, 0);
> - else
> + } else {
> tcg_gen_mov_tl(t, cpu_gpr[reg]);
> + }
> }
>
> static inline void gen_store_gpr(TCGv t, int reg)
> {
> - if (reg != 0)
> + if (reg != 0) {
> tcg_gen_mov_tl(cpu_gpr[reg], t);
> + }
> }
>
> /* Moves to/from shadow registers. */
> @@ -2636,9 +2638,9 @@ static inline void gen_load_srsgpr(int from, int to)
> {
> TCGv t0 = tcg_temp_new();
>
> - if (from == 0)
> + if (from == 0) {
> tcg_gen_movi_tl(t0, 0);
> - else {
> + } else {
> TCGv_i32 t2 = tcg_temp_new_i32();
> TCGv_ptr addr = tcg_temp_new_ptr();
>
> @@ -2841,10 +2843,11 @@ static void gen_store_fpr64(DisasContext *ctx, TCGv_i64 t, int reg)
>
> static inline int get_fp_bit(int cc)
> {
> - if (cc)
> + if (cc) {
> return 24 + cc;
> - else
> + } else {
> return 23;
> + }
> }
>
> /* Addresses computation */
> @@ -2908,14 +2911,16 @@ static inline void gen_move_high32(TCGv ret, TCGv_i64 arg)
>
> static inline void check_cp0_enabled(DisasContext *ctx)
> {
> - if (unlikely(!(ctx->hflags & MIPS_HFLAG_CP0)))
> + if (unlikely(!(ctx->hflags & MIPS_HFLAG_CP0))) {
> generate_exception_err(ctx, EXCP_CpU, 0);
> + }
> }
>
> static inline void check_cp1_enabled(DisasContext *ctx)
> {
> - if (unlikely(!(ctx->hflags & MIPS_HFLAG_FPU)))
> + if (unlikely(!(ctx->hflags & MIPS_HFLAG_FPU))) {
> generate_exception_err(ctx, EXCP_CpU, 1);
> + }
> }
>
> /* Verify that the processor is running with COP1X instructions enabled.
> @@ -2924,8 +2929,9 @@ static inline void check_cp1_enabled(DisasContext *ctx)
>
> static inline void check_cop1x(DisasContext *ctx)
> {
> - if (unlikely(!(ctx->hflags & MIPS_HFLAG_COP1X)))
> + if (unlikely(!(ctx->hflags & MIPS_HFLAG_COP1X))) {
> generate_exception_end(ctx, EXCP_RI);
> + }
> }
>
> /* Verify that the processor is running with 64-bit floating-point
> @@ -2933,8 +2939,9 @@ static inline void check_cop1x(DisasContext *ctx)
>
> static inline void check_cp1_64bitmode(DisasContext *ctx)
> {
> - if (unlikely(~ctx->hflags & (MIPS_HFLAG_F64 | MIPS_HFLAG_COP1X)))
> + if (unlikely(~ctx->hflags & (MIPS_HFLAG_F64 | MIPS_HFLAG_COP1X))) {
> generate_exception_end(ctx, EXCP_RI);
> + }
> }
>
> /*
> @@ -2950,8 +2957,9 @@ static inline void check_cp1_64bitmode(DisasContext *ctx)
> */
> static inline void check_cp1_registers(DisasContext *ctx, int regs)
> {
> - if (unlikely(!(ctx->hflags & MIPS_HFLAG_F64) && (regs & 1)))
> + if (unlikely(!(ctx->hflags & MIPS_HFLAG_F64) && (regs & 1))) {
> generate_exception_end(ctx, EXCP_RI);
> + }
> }
>
> /* Verify that the processor is running with DSP instructions enabled.
> @@ -3040,8 +3048,9 @@ static inline void check_ps(DisasContext *ctx)
> instructions are not enabled. */
> static inline void check_mips_64(DisasContext *ctx)
> {
> - if (unlikely(!(ctx->hflags & MIPS_HFLAG_64)))
> + if (unlikely(!(ctx->hflags & MIPS_HFLAG_64))) {
> generate_exception_end(ctx, EXCP_RI);
> + }
> }
> #endif
>
> @@ -3131,8 +3140,7 @@ static inline void check_nms_dl_il_sl_tl_l2c(DisasContext *ctx)
> !(ctx->CP0_Config1 & (1 << CP0C1_IL)) &&
> !(ctx->CP0_Config2 & (1 << CP0C2_SL)) &&
> !(ctx->CP0_Config2 & (1 << CP0C2_TL)) &&
> - !(ctx->CP0_Config5 & (1 << CP0C5_L2C)))
> - {
> + !(ctx->CP0_Config5 & (1 << CP0C5_L2C))) {
> generate_exception_end(ctx, EXCP_RI);
> }
> }
> @@ -3882,22 +3890,25 @@ static void gen_logic_imm(DisasContext *ctx, uint32_t opc,
> uimm = (uint16_t)imm;
> switch (opc) {
> case OPC_ANDI:
> - if (likely(rs != 0))
> + if (likely(rs != 0)) {
> tcg_gen_andi_tl(cpu_gpr[rt], cpu_gpr[rs], uimm);
> - else
> + } else {
> tcg_gen_movi_tl(cpu_gpr[rt], 0);
> + }
> break;
> case OPC_ORI:
> - if (rs != 0)
> + if (rs != 0) {
> tcg_gen_ori_tl(cpu_gpr[rt], cpu_gpr[rs], uimm);
> - else
> + } else {
> tcg_gen_movi_tl(cpu_gpr[rt], uimm);
> + }
> break;
> case OPC_XORI:
> - if (likely(rs != 0))
> + if (likely(rs != 0)) {
> tcg_gen_xori_tl(cpu_gpr[rt], cpu_gpr[rs], uimm);
> - else
> + } else {
> tcg_gen_movi_tl(cpu_gpr[rt], uimm);
> + }
> break;
> case OPC_LUI:
> if (rs != 0 && (ctx->insn_flags & ISA_MIPS32R6)) {
> @@ -6060,8 +6071,9 @@ static void gen_compute_branch (DisasContext *ctx, uint32_t opc,
> }
>
> out:
> - if (insn_bytes == 2)
> + if (insn_bytes == 2) {
> ctx->hflags |= MIPS_HFLAG_B16;
> + }
> tcg_temp_free(t0);
> tcg_temp_free(t1);
> }
> @@ -6708,8 +6720,9 @@ static void gen_mfc0(DisasContext *ctx, TCGv arg, int reg, int sel)
> {
> const char *register_name = "invalid";
>
> - if (sel != 0)
> + if (sel != 0) {
> check_insn(ctx, ISA_MIPS32);
> + }
>
> switch (reg) {
> case CP0_REGISTER_00:
> @@ -7464,8 +7477,9 @@ static void gen_mtc0(DisasContext *ctx, TCGv arg, int reg, int sel)
> {
> const char *register_name = "invalid";
>
> - if (sel != 0)
> + if (sel != 0) {
> check_insn(ctx, ISA_MIPS32);
> + }
>
> if (tb_cflags(ctx->base.tb) & CF_USE_ICOUNT) {
> gen_io_start();
> @@ -8210,8 +8224,9 @@ static void gen_dmfc0(DisasContext *ctx, TCGv arg, int reg, int sel)
> {
> const char *register_name = "invalid";
>
> - if (sel != 0)
> + if (sel != 0) {
> check_insn(ctx, ISA_MIPS64);
> + }
>
> switch (reg) {
> case CP0_REGISTER_00:
> @@ -8920,8 +8935,9 @@ static void gen_dmtc0(DisasContext *ctx, TCGv arg, int reg, int sel)
> {
> const char *register_name = "invalid";
>
> - if (sel != 0)
> + if (sel != 0) {
> check_insn(ctx, ISA_MIPS64);
> + }
>
> if (tb_cflags(ctx->base.tb) & CF_USE_ICOUNT) {
> gen_io_start();
> @@ -10162,8 +10178,9 @@ static void gen_cp0(CPUMIPSState *env, DisasContext *ctx, uint32_t opc,
> break;
> case OPC_TLBWI:
> opn = "tlbwi";
> - if (!env->tlb->helper_tlbwi)
> + if (!env->tlb->helper_tlbwi) {
> goto die;
> + }
> gen_helper_tlbwi(cpu_env);
> break;
> case OPC_TLBINV:
> @@ -10186,20 +10203,23 @@ static void gen_cp0(CPUMIPSState *env, DisasContext *ctx, uint32_t opc,
> break;
> case OPC_TLBWR:
> opn = "tlbwr";
> - if (!env->tlb->helper_tlbwr)
> + if (!env->tlb->helper_tlbwr) {
> goto die;
> + }
> gen_helper_tlbwr(cpu_env);
> break;
> case OPC_TLBP:
> opn = "tlbp";
> - if (!env->tlb->helper_tlbp)
> + if (!env->tlb->helper_tlbp) {
> goto die;
> + }
> gen_helper_tlbp(cpu_env);
> break;
> case OPC_TLBR:
> opn = "tlbr";
> - if (!env->tlb->helper_tlbr)
> + if (!env->tlb->helper_tlbr) {
> goto die;
> + }
> gen_helper_tlbr(cpu_env);
> break;
> case OPC_ERET: /* OPC_ERETNC */
> @@ -10273,8 +10293,9 @@ static void gen_compute_branch1(DisasContext *ctx, uint32_t op,
> goto out;
> }
>
> - if (cc != 0)
> + if (cc != 0) {
> check_insn(ctx, ISA_MIPS4 | ISA_MIPS32);
> + }
>
> btarget = ctx->base.pc_next + 4 + offset;
>
> @@ -10728,10 +10749,11 @@ static void gen_movci(DisasContext *ctx, int rd, int rs, int cc, int tf)
> return;
> }
>
> - if (tf)
> + if (tf) {
> cond = TCG_COND_EQ;
> - else
> + } else {
> cond = TCG_COND_NE;
> + }
>
> l1 = gen_new_label();
> t0 = tcg_temp_new_i32();
> @@ -10753,10 +10775,11 @@ static inline void gen_movcf_s(DisasContext *ctx, int fs, int fd, int cc,
> TCGv_i32 t0 = tcg_temp_new_i32();
> TCGLabel *l1 = gen_new_label();
>
> - if (tf)
> + if (tf) {
> cond = TCG_COND_EQ;
> - else
> + } else {
> cond = TCG_COND_NE;
> + }
>
> tcg_gen_andi_i32(t0, fpu_fcr31, 1 << get_fp_bit(cc));
> tcg_gen_brcondi_i32(cond, t0, 0, l1);
> @@ -10774,10 +10797,11 @@ static inline void gen_movcf_d(DisasContext *ctx, int fs, int fd, int cc,
> TCGv_i64 fp0;
> TCGLabel *l1 = gen_new_label();
>
> - if (tf)
> + if (tf) {
> cond = TCG_COND_EQ;
> - else
> + } else {
> cond = TCG_COND_NE;
> + }
>
> tcg_gen_andi_i32(t0, fpu_fcr31, 1 << get_fp_bit(cc));
> tcg_gen_brcondi_i32(cond, t0, 0, l1);
> @@ -10797,10 +10821,11 @@ static inline void gen_movcf_ps(DisasContext *ctx, int fs, int fd,
> TCGLabel *l1 = gen_new_label();
> TCGLabel *l2 = gen_new_label();
>
> - if (tf)
> + if (tf) {
> cond = TCG_COND_EQ;
> - else
> + } else {
> cond = TCG_COND_NE;
> + }
>
> tcg_gen_andi_i32(t0, fpu_fcr31, 1 << get_fp_bit(cc));
> tcg_gen_brcondi_i32(cond, t0, 0, l1);
> @@ -12096,8 +12121,9 @@ static void gen_farith(DisasContext *ctx, enum fopcode op1,
> TCGLabel *l1 = gen_new_label();
> TCGv_i64 fp0;
>
> - if (ft != 0)
> + if (ft != 0) {
> tcg_gen_brcondi_tl(TCG_COND_NE, cpu_gpr[ft], 0, l1);
> + }
> fp0 = tcg_temp_new_i64();
> gen_load_fpr64(ctx, fp0, fs);
> gen_store_fpr64(ctx, fp0, fd);
>
prev parent reply other threads:[~2019-06-20 11:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-20 9:19 [Qemu-devel] [PATCH 0/4] target/mips: Misc fixes and maintenance for 4.1 Aleksandar Markovic
2019-06-20 9:19 ` [Qemu-devel] [PATCH 1/4] MAINTAINERS: Update file items for MIPS Malta board Aleksandar Markovic
2019-06-20 11:13 ` Philippe Mathieu-Daudé
2019-06-20 9:19 ` [Qemu-devel] [PATCH 2/4] MAINTAINERS: Consolidate MIPS disassembler-related items Aleksandar Markovic
2019-06-20 9:19 ` [Qemu-devel] [PATCH 3/4] target/mips: Fix some space checkpatch errors in translate.c Aleksandar Markovic
2019-06-20 11:11 ` Philippe Mathieu-Daudé
2019-06-20 9:19 ` [Qemu-devel] [PATCH 4/4] target/mips: Fix if-else arms " Aleksandar Markovic
2019-06-20 11:11 ` Philippe Mathieu-Daudé [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a1ac2dbf-9c7e-24a2-d17b-99cbe08d6640@redhat.com \
--to=philmd@redhat.com \
--cc=aleksandar.markovic@rt-rk.com \
--cc=amarkovic@wavecomp.com \
--cc=arikalo@wavecomp.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.