Re: [PATCH v2 3/6] tcg/ppc: Hoist common argument loads in tcg_out_op()

2021-01-14 Thread Philippe Mathieu-Daudé
On 1/14/21 1:28 AM, Richard Henderson wrote: > > Let's just drop the hoisting parts and only do the signature parts for now. > I'd rather think of a way to split up this large function than waste time > optimizing it. Agreed :) Thanks!

Re: [PATCH v2 3/6] tcg/ppc: Hoist common argument loads in tcg_out_op()

2021-01-13 Thread Richard Henderson
On 1/13/21 7:24 AM, Philippe Mathieu-Daudé wrote: > case INDEX_op_ld8s_i32: > case INDEX_op_ld8s_i64: > -tcg_out_mem_long(s, LBZ, LBZX, args[0], args[1], args[2]); > +tcg_out_mem_long(s, LBZ, LBZX, a0, a1, a2); > tcg_out32(s, EXTSB | RS(args[0]) | RA(args[0])); M

[PATCH v2 3/6] tcg/ppc: Hoist common argument loads in tcg_out_op()

2021-01-13 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé --- tcg/ppc/tcg-target.c.inc | 188 ++- 1 file changed, 85 insertions(+), 103 deletions(-) diff --git a/tcg/ppc/tcg-target.c.inc b/tcg/ppc/tcg-target.c.inc index 19a4a12f155..70b747a8a30 100644 --- a/tcg/ppc/tcg-target.c.i