[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/3] target/hppa: sub: fix trap on overflow for narrow mode
From: |
Sven Schnelle |
Subject: |
Re: [PATCH 2/3] target/hppa: sub: fix trap on overflow for narrow mode |
Date: |
Thu, 21 Mar 2024 21:08:29 +0100 |
Richard Henderson <richard.henderson@linaro.org> writes:
> On 3/21/24 08:42, Sven Schnelle wrote:
>> Signed-off-by: Sven Schnelle <svens@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.henderson@linaro.org>
Thanks!