target/hppa: Always make a copy in do_ibranch_priv

This simplifies callers, which might otherwise have
to make another copy.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
This commit is contained in:
Richard Henderson 2024-03-27 12:50:07 -10:00
parent 0bb0202962
commit 1874e6c2fd

View file

@ -1968,18 +1968,17 @@ static bool do_ibranch(DisasContext *ctx, TCGv_i64 dest, TCGv_i64 dspc,
*/ */
static TCGv_i64 do_ibranch_priv(DisasContext *ctx, TCGv_i64 offset) static TCGv_i64 do_ibranch_priv(DisasContext *ctx, TCGv_i64 offset)
{ {
TCGv_i64 dest; TCGv_i64 dest = tcg_temp_new_i64();
switch (ctx->privilege) { switch (ctx->privilege) {
case 0: case 0:
/* Privilege 0 is maximum and is allowed to decrease. */ /* Privilege 0 is maximum and is allowed to decrease. */
return offset; tcg_gen_mov_i64(dest, offset);
break;
case 3: case 3:
/* Privilege 3 is minimum and is never allowed to increase. */ /* Privilege 3 is minimum and is never allowed to increase. */
dest = tcg_temp_new_i64();
tcg_gen_ori_i64(dest, offset, 3); tcg_gen_ori_i64(dest, offset, 3);
break; break;
default: default:
dest = tcg_temp_new_i64();
tcg_gen_andi_i64(dest, offset, -4); tcg_gen_andi_i64(dest, offset, -4);
tcg_gen_ori_i64(dest, dest, ctx->privilege); tcg_gen_ori_i64(dest, dest, ctx->privilege);
tcg_gen_umax_i64(dest, dest, offset); tcg_gen_umax_i64(dest, dest, offset);