[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 9/9] target/mips: Explode gen_msa_branch() as gen_msa_BxZ_V/B
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 9/9] target/mips: Explode gen_msa_branch() as gen_msa_BxZ_V/BxZ() |
Date: |
Fri, 4 Dec 2020 23:53:49 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
On 12/4/20 6:04 PM, Richard Henderson wrote:
> On 12/2/20 12:44 PM, Philippe Mathieu-Daudé wrote:
>> +static bool gen_msa_BxZ(DisasContext *ctx, int df, int wt, int s16, bool
>> if_not)
>> +{
>> + check_msa_access(ctx);
>> +
>> + if (ctx->hflags & MIPS_HFLAG_BMASK) {
>> + generate_exception_end(ctx, EXCP_RI);
>> + return true;
>> + }
>> +
>> + gen_check_zero_element(bcond, df, wt);
>> + if (if_not) {
>> + tcg_gen_setcondi_tl(TCG_COND_EQ, bcond, bcond, 0);
>> + }
>
> Since gen_check_zero_element already produces a boolean, this is better as
>
> tcg_gen_xori_tl(bcond, bcond, if_not);
>
> where tcg_gen_xori_tl already contains the if.
Ah, got it.
>
>> case OPC_BNZ_D:
>> - gen_check_zero_element(bcond, df, wt);
>> - tcg_gen_setcondi_tl(TCG_COND_EQ, bcond, bcond, 0);
>> + gen_msa_BxZ(ctx, df, wt, s16, true);
>
> ... oops, that'd be for a follow-up patch, to make this patch just code
> movement.
Yes, will follow. I'm tempted to inline gen_check_zero_element (actually
move gen_msa_BxZ as gen_check_zero_element prologue/epilogue). Not sure
gen_check_zero_element() can be reused later though.
>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Thanks!
>
> r~
>
- Re: [PATCH 6/9] target/mips: Alias MSA vector registers on FPU scalar registers, (continued)
[PATCH 7/9] target/mips: Extract msa_translate_init() from mips_tcg_init(), Philippe Mathieu-Daudé, 2020/12/02
[PATCH 8/9] target/mips: Remove CPUMIPSState* argument from gen_msa*() methods, Philippe Mathieu-Daudé, 2020/12/02
[PATCH 9/9] target/mips: Explode gen_msa_branch() as gen_msa_BxZ_V/BxZ(), Philippe Mathieu-Daudé, 2020/12/02
Re: [PATCH 0/9] target/mips: Simplify MSA TCG logic, Jiaxun Yang, 2020/12/02
Re: [PATCH 0/9] target/mips: Simplify MSA TCG logic, Jiaxun Yang, 2020/12/02