Re: [3/3] Record the number of generator arguments in insn_data

2011-04-01 Thread Richard Sandiford
Richard Henderson writes: > On 03/31/2011 01:09 PM, Richard Sandiford wrote: >>> I think the assert should be retained (for now) as >>> >>> gcc_assert (nops == 4 || nops == 6); >>> >>> at least until we add such verification to some genfoo. >> >> After the patch we have: >> >> gcc_assert (no

Re: [3/3] Record the number of generator arguments in insn_data

2011-03-31 Thread Richard Henderson
On 03/31/2011 01:09 PM, Richard Sandiford wrote: >> I think the assert should be retained (for now) as >> >> gcc_assert (nops == 4 || nops == 6); >> >> at least until we add such verification to some genfoo. > > After the patch we have: > > gcc_assert (nops == (unsigned int) insn_data[(int) i

Re: [3/3] Record the number of generator arguments in insn_data

2011-03-31 Thread Richard Sandiford
Richard Henderson writes: > On 03/31/2011 09:23 AM, Richard Sandiford wrote: >> +++ gcc/expr.c 2011-03-31 16:49:06.0 +0100 >> @@ -1293,11 +1293,7 @@ emit_block_move_via_movmem (rtx x, rtx y >> nice if there were some way to inform the backend, so >> that it doesn'

Re: [3/3] Record the number of generator arguments in insn_data

2011-03-31 Thread Richard Henderson
On 03/31/2011 09:23 AM, Richard Sandiford wrote: > +++ gcc/expr.c2011-03-31 16:49:06.0 +0100 > @@ -1293,11 +1293,7 @@ emit_block_move_via_movmem (rtx x, rtx y >nice if there were some way to inform the backend, so >that it doesn't fail the expansion because i

[3/3] Record the number of generator arguments in insn_data

2011-03-31 Thread Richard Sandiford
This patch makes use of the new n_generator_args field. Bboostrapped & regressions-tested on x86_64-linux-gnu. Also tested by building cc1 for arm-linux-gnueabi, mips-elf, spu-elf. Richard gcc/ * expr.c (emit_block_move_via_movmem): Use n_generator_args instead of n_operands.