[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 23/25] target/tricore: Use min/max for saturate
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v2 23/25] target/tricore: Use min/max for saturate |
Date: |
Thu, 9 Mar 2023 11:09:05 +0100 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 |
On 7/3/23 19:35, Richard Henderson wrote:
Use tcg_constant_i32 for the bounds.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
Cc: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
---
target/tricore/translate.c | 14 +++-----------
1 file changed, 3 insertions(+), 11 deletions(-)
static void gen_saturate(TCGv ret, TCGv arg, int32_t up, int32_t low)
{
- TCGv sat_neg = tcg_const_i32(low);
- TCGv temp = tcg_const_i32(up);
-
- /* sat_neg = (arg < low ) ? low : arg; */
- tcg_gen_movcond_tl(TCG_COND_LT, sat_neg, arg, sat_neg, sat_neg, arg);
-
- /* ret = (sat_neg > up ) ? up : sat_neg; */
- tcg_gen_movcond_tl(TCG_COND_GT, ret, sat_neg, temp, temp, sat_neg);
+ tcg_gen_smax_tl(ret, arg, tcg_constant_i32(low));
This one is trivial when looking at tcg_gen_smax implementation.
+ tcg_gen_smin_tl(ret, ret, tcg_constant_i32(up));
OK, when changing TCG_COND_GT -> TCG_COND_LT and inverting the args,
then this becomes obvious this is tcg_gen_smin.
}
static void gen_saturate_u(TCGv ret, TCGv arg, int32_t up)
{
- TCGv temp = tcg_const_i32(up);
- /* sat_neg = (arg > up ) ? up : arg; */
- tcg_gen_movcond_tl(TCG_COND_GTU, ret, arg, temp, temp, arg);
+ tcg_gen_umin_tl(ret, arg, tcg_constant_i32(up));
Inverting args for TCG_COND_GTU -> TCG_COND_LTU, this is indeed
tcg_gen_umin.
}
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
- Re: [PATCH v2 19/25] target/ppc: Avoid tcg_const_* in power8-pmu-regs.c.inc, (continued)
- [PATCH v2 21/25] target/ppc: Fix gen_tlbsx_booke206, Richard Henderson, 2023/03/07
- [PATCH v2 22/25] target/ppc: Avoid tcg_const_* in translate.c, Richard Henderson, 2023/03/07
- [PATCH v2 20/25] target/ppc: Rewrite trans_ADDG6S, Richard Henderson, 2023/03/07
- [PATCH v2 23/25] target/tricore: Use min/max for saturate, Richard Henderson, 2023/03/07
- Re: [PATCH v2 23/25] target/tricore: Use min/max for saturate,
Philippe Mathieu-Daudé <=
- [PATCH v2 25/25] tcg: Drop tcg_const_*, Richard Henderson, 2023/03/07
- [PATCH v2 24/25] tcg: Drop tcg_const_*_vec, Richard Henderson, 2023/03/07