|
From: | Daniel Henrique Barboza |
Subject: | Re: [PATCH v2 12/54] tcg/riscv: Conditionalize tcg_out_exts_i32_i64 |
Date: | Wed, 12 Apr 2023 17:01:00 -0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.1 |
On 4/10/23 22:04, Richard Henderson wrote:
Since TCG_TYPE_I32 values are kept sign-extended in registers, via "w" instructions, we need not extend if the register matches.
Perhaps "we don't need to extend if ..." ?
This is already relied upon by comparisons. Signed-off-by: Richard Henderson <richard.henderson@linaro.org> ---
Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
tcg/riscv/tcg-target.c.inc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tcg/riscv/tcg-target.c.inc b/tcg/riscv/tcg-target.c.inc index 7bd3b421ad..2b9aab29ec 100644 --- a/tcg/riscv/tcg-target.c.inc +++ b/tcg/riscv/tcg-target.c.inc @@ -604,7 +604,9 @@ static void tcg_out_ext32s(TCGContext *s, TCGReg ret, TCGReg arg)static void tcg_out_exts_i32_i64(TCGContext *s, TCGReg ret, TCGReg arg){ - tcg_out_ext32s(s, ret, arg); + if (ret != arg) { + tcg_out_ext32s(s, ret, arg); + } }static void tcg_out_ldst(TCGContext *s, RISCVInsn opc, TCGReg data,
[Prev in Thread] | Current Thread | [Next in Thread] |