On 10/9/25 03:10, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> target/hppa/translate.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/target/hppa/translate.c b/target/hppa/translate.c
> index 3f78bb2a76a..92071affe47 100644
> --- a/target/hppa/translate.c
> +++ b/target/hppa/translate.c
> @@ -1637,6 +1637,7 @@ static void do_store_32(DisasContext *ctx, TCGv_i32 src, unsigned rb,
> /* Caller uses nullify_over/nullify_end. */
> assert(ctx->null_cond.c == TCG_COND_NEVER);
>
> + mop |= MO_TE;
> form_gva(ctx, &addr, &ofs, rb, rx, scale, disp, sp, modify,
> MMU_DISABLED(ctx));
> tcg_gen_qemu_st_i32(src, addr, ctx->mmu_idx, mop | UNALIGN(ctx));
> @@ -1655,6 +1656,7 @@ static void do_store_64(DisasContext *ctx, TCGv_i64 src, unsigned rb,
> /* Caller uses nullify_over/nullify_end. */
> assert(ctx->null_cond.c == TCG_COND_NEVER);
>
> + mop |= MO_TE;
> form_gva(ctx, &addr, &ofs, rb, rx, scale, disp, sp, modify,
> MMU_DISABLED(ctx));
> tcg_gen_qemu_st_i64(src, addr, ctx->mmu_idx, mop | UNALIGN(ctx));
> @@ -1739,7 +1741,7 @@ static bool do_store(DisasContext *ctx, unsigned rt, unsigned rb,
> int modify, MemOp mop)
> {
> nullify_over(ctx);
> - do_store_64(ctx, load_gpr(ctx, rt), rb, 0, 0, disp, sp, modify, MO_TE | mop);
> + do_store_64(ctx, load_gpr(ctx, rt), rb, 0, 0, disp, sp, modify, mop);
> return nullify_end(ctx);
> }
>
> @@ -1752,7 +1754,7 @@ static bool do_fstorew(DisasContext *ctx, unsigned rt, unsigned rb,
> nullify_over(ctx);
>
> tmp = load_frw_i32(rt);
> - do_store_32(ctx, tmp, rb, rx, scale, disp, sp, modify, MO_TE | MO_UL);
> + do_store_32(ctx, tmp, rb, rx, scale, disp, sp, modify, MO_UL);
>
> return nullify_end(ctx);
> }
> @@ -1772,7 +1774,7 @@ static bool do_fstored(DisasContext *ctx, unsigned rt, unsigned rb,
> nullify_over(ctx);
>
> tmp = load_frd(rt);
> - do_store_64(ctx, tmp, rb, rx, scale, disp, sp, modify, MO_TE | MO_UQ);
> + do_store_64(ctx, tmp, rb, rx, scale, disp, sp, modify, MO_UQ);
>
> return nullify_end(ctx);
> }
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
r~