[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 11/42] target/mips/tx79: Move PCPYH opcode to decodetree
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [RFC PATCH 11/42] target/mips/tx79: Move PCPYH opcode to decodetree |
Date: |
Mon, 8 Mar 2021 12:57:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0 |
On 3/8/21 11:48 AM, Philippe Mathieu-Daudé wrote:
> On 2/15/21 5:26 PM, Richard Henderson wrote:
>> On 2/14/21 9:58 AM, Philippe Mathieu-Daudé wrote:
>>> + if (a->rt == 0) {
>>> + tcg_gen_movi_i64(cpu_gpr[a->rd], 0);
>>> + tcg_gen_movi_i64(cpu_gpr_hi[a->rd], 0);
>>> + return true;
>>> + }
>>
>> Is there a good reason not to use gen_load_gpr?
>
> I suppose you meant gen_store_gpr*().
Double checking, we check for $rt, not $rd, right?
>
> We need to check $rt anyway to not do the deposit calls
> if it is non-zero.
>
> As it is mostly code movement, I prefer to keep it as it
> for now, we might improve it later.
>
>>
>> Otherwise,
>> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
>
> Thanks!
>