Richard Henderson <richard.hender...@linaro.org> writes: > On 3/21/24 08:42, Sven Schnelle wrote: >> Signed-off-by: Sven Schnelle <sv...@stackframe.org> >> --- >> target/hppa/translate.c | 3 +++ >> 1 file changed, 3 insertions(+) >> diff --git a/target/hppa/translate.c b/target/hppa/translate.c >> index f493e207e1..4d2b96f876 100644 >> --- a/target/hppa/translate.c >> +++ b/target/hppa/translate.c >> @@ -1213,6 +1213,9 @@ static void do_sub(DisasContext *ctx, unsigned rt, >> TCGv_i64 in1, >> if (is_tsv || cond_need_sv(c)) { >> sv = do_sub_sv(ctx, dest, in1, in2); >> if (is_tsv) { >> + if (!d) { >> + tcg_gen_ext32s_i64(sv, sv); >> + } >> gen_helper_tsv(tcg_env, sv); >> } >> } > > Difficult to pinpoint exactly which patch should have added this. :-)
Yes, after missing the Fixes: tags on all of my patches in the last patchset, i tried add one but wasn't sure either. :-) > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> Thanks!