[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 12/24] tcg/aarch64: Implement negsetcond_*
From: |
Peter Maydell |
Subject: |
Re: [PATCH 12/24] tcg/aarch64: Implement negsetcond_* |
Date: |
Thu, 10 Aug 2023 17:58:08 +0100 |
On Thu, 10 Aug 2023 at 17:55, Richard Henderson
<richard.henderson@linaro.org> wrote:
>
> On 8/10/23 09:39, Peter Maydell wrote:
> >> + case INDEX_op_negsetcond_i32:
> >> + a2 = (int32_t)a2;
> >> + /* FALLTHRU */
> >
> > I see this is what we already do for setcond and movcond,
> > but how does it work when the 2nd input is a register?
> > Or is reg-reg guaranteed to always use the _i64 op?
>
> For reg-reg, a2 < 31, so the sign-extend does nothing.
OK. Do we document somewhere what a TCGArg is?
Anyway,
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
thanks
-- PMM
- [PATCH 09/24] target/tricore: Replace gen_cond_w with tcg_gen_negsetcond_tl, (continued)
[PATCH 13/24] tcg/arm: Implement negsetcond_i32, Richard Henderson, 2023/08/07
[PATCH 14/24] tcg/riscv: Implement negsetcond_*, Richard Henderson, 2023/08/07
[PATCH 15/24] tcg/s390x: Implement negsetcond_*, Richard Henderson, 2023/08/07
[PATCH 16/24] tcg/sparc64: Implement negsetcond_*, Richard Henderson, 2023/08/07
[PATCH 17/24] tcg/i386: Merge tcg_out_brcond{32,64}, Richard Henderson, 2023/08/07