[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 10/19] target-mips: add MSA I5 format instruc
From: |
Leon Alrae |
Subject: |
Re: [Qemu-devel] [PATCH v3 10/19] target-mips: add MSA I5 format instruction |
Date: |
Sun, 2 Nov 2014 17:53:44 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111124 Thunderbird/8.0 |
Hi Yongbok,
On 01/11/14 05:28, Yongbok Kim wrote:
> + TCGv_i32 timm = tcg_temp_new_i32();
> + tcg_gen_movi_i32(timm, u5);
> +
> + switch (MASK_MSA_I5(ctx->opcode)) {
> + case OPC_ADDVI_df:
> + gen_helper_msa_addvi_df(cpu_env, tdf, twd, tws, timm);
> + break;
> + case OPC_SUBVI_df:
> + gen_helper_msa_subvi_df(cpu_env, tdf, twd, tws, timm);
> + break;
> + case OPC_MAXI_S_df:
> + tcg_gen_movi_i32(timm, s5);
> + gen_helper_msa_maxi_s_df(cpu_env, tdf, twd, tws, timm);
I think tcg_gen_movi_i32(timm, u5) shouldn't be in front of the switch,
this operation is useless for instructions having signed immediate.
Probably it's not a big deal as tcg will optimize it out, but it needs
to be corrected at some point (the same applies to the 3RF group
introduced in patch #14). Otherwise,
Reviewed-by: Leon Alrae <address@hidden>
- [Qemu-devel] [PATCH v3 01/19] target-mips: add MSA defines and data structure, (continued)
- [Qemu-devel] [PATCH v3 01/19] target-mips: add MSA defines and data structure, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 02/19] target-mips: add MSA exceptions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 03/19] target-mips: remove duplicated mips/ieee mapping function, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 04/19] target-mips: stop translation after ctc1, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 05/19] target-mips: add MSA opcode enum, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 06/19] target-mips: add msa_reset(), global msa register, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 07/19] target-mips: add msa_helper.c, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 08/19] target-mips: add MSA branch instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 09/19] target-mips: add MSA I8 format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 10/19] target-mips: add MSA I5 format instruction, Yongbok Kim, 2014/11/01
- Re: [Qemu-devel] [PATCH v3 10/19] target-mips: add MSA I5 format instruction,
Leon Alrae <=
- [Qemu-devel] [PATCH v3 11/19] target-mips: add MSA BIT format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 12/19] target-mips: add MSA 3R format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 13/19] target-mips: add MSA ELM format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 14/19] target-mips: add MSA 3RF format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 15/19] target-mips: add MSA VEC/2R format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 16/19] target-mips: add MSA 2RF format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 17/19] target-mips: add MSA MI10 format instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 18/19] disas/mips.c: disassemble MSA instructions, Yongbok Kim, 2014/11/01
- [Qemu-devel] [PATCH v3 19/19] target-mips: add MSA support to mips32r5-generic, Yongbok Kim, 2014/11/01