On Wed Apr 16, 2025 at 5:23 AM AEST, Richard Henderson wrote: > Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
Looks equivalent. Reviewed-by: Nicholas Piggin <npig...@gmail.com> > --- > tcg/ppc/tcg-target.c.inc | 21 +++++++-------------- > 1 file changed, 7 insertions(+), 14 deletions(-) > > diff --git a/tcg/ppc/tcg-target.c.inc b/tcg/ppc/tcg-target.c.inc > index 1782d05290..669c5eae4a 100644 > --- a/tcg/ppc/tcg-target.c.inc > +++ b/tcg/ppc/tcg-target.c.inc > @@ -2206,8 +2206,8 @@ static void tcg_out_cntxz(TCGContext *s, TCGType type, > uint32_t opc, > } > } > > -static void tcg_out_cmp2(TCGContext *s, const TCGArg *args, > - const int *const_args) > +static void tcg_out_cmp2(TCGContext *s, TCGCond cond, TCGReg al, TCGReg ah, > + TCGArg bl, bool blconst, TCGArg bh, bool bhconst) > { > static const struct { uint8_t bit1, bit2; } bits[] = { > [TCG_COND_LT ] = { CR_LT, CR_LT }, > @@ -2220,18 +2220,9 @@ static void tcg_out_cmp2(TCGContext *s, const TCGArg > *args, > [TCG_COND_GEU] = { CR_GT, CR_LT }, > }; > > - TCGCond cond = args[4], cond2; > - TCGArg al, ah, bl, bh; > - int blconst, bhconst; > + TCGCond cond2; > int op, bit1, bit2; > > - al = args[0]; > - ah = args[1]; > - bl = args[2]; > - bh = args[3]; > - blconst = const_args[2]; > - bhconst = const_args[3]; > - > switch (cond) { > case TCG_COND_EQ: > op = CRAND; > @@ -2286,7 +2277,8 @@ static void tcg_out_cmp2(TCGContext *s, const TCGArg > *args, > static void tcg_out_setcond2(TCGContext *s, const TCGArg *args, > const int *const_args) > { > - tcg_out_cmp2(s, args + 1, const_args + 1); > + tcg_out_cmp2(s, args[5], args[1], args[2], args[3], const_args[3], > + args[4], const_args[4]); > tcg_out32(s, MFOCRF | RT(TCG_REG_R0) | FXM(0)); > tcg_out_rlw(s, RLWINM, args[0], TCG_REG_R0, CR_EQ + 0*4 + 1, 31, 31); > } > @@ -2294,7 +2286,8 @@ static void tcg_out_setcond2(TCGContext *s, const > TCGArg *args, > static void tcg_out_brcond2(TCGContext *s, const TCGArg *args, > const int *const_args) > { > - tcg_out_cmp2(s, args, const_args); > + tcg_out_cmp2(s, args[4], args[0], args[1], args[2], const_args[2], > + args[3], const_args[3]); > tcg_out_bc_lab(s, TCG_COND_EQ, arg_label(args[5])); > } >