Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread Andreas Krebbel
On 03/01/2016 02:38 PM, James Greenhalgh wrote: > On Tue, Mar 01, 2016 at 01:35:18PM +0100, Andreas Krebbel wrote: >> On 03/01/2016 01:15 PM, James Greenhalgh wrote: >>> On Tue, Mar 01, 2016 at 10:29:28AM +0100, Andreas Krebbel wrote: On 02/29/2016 02:36 PM, Bernd Schmidt wrote: > On 02/29

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread James Greenhalgh
On Tue, Mar 01, 2016 at 01:35:18PM +0100, Andreas Krebbel wrote: > On 03/01/2016 01:15 PM, James Greenhalgh wrote: > > On Tue, Mar 01, 2016 at 10:29:28AM +0100, Andreas Krebbel wrote: > >> On 02/29/2016 02:36 PM, Bernd Schmidt wrote: > >>> On 02/29/2016 09:46 AM, Andreas Krebbel wrote: > Ok fo

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread Andreas Krebbel
On 03/01/2016 01:15 PM, James Greenhalgh wrote: > On Tue, Mar 01, 2016 at 10:29:28AM +0100, Andreas Krebbel wrote: >> On 02/29/2016 02:36 PM, Bernd Schmidt wrote: >>> On 02/29/2016 09:46 AM, Andreas Krebbel wrote: Ok for mainline? * gensupport.c (process_substs_on_one_elem): Split

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread Bernd Schmidt
On 03/01/2016 01:15 PM, James Greenhalgh wrote: On Tue, Mar 01, 2016 at 10:29:28AM +0100, Andreas Krebbel wrote: On 02/29/2016 02:36 PM, Bernd Schmidt wrote: Didn't I approve this a while ago? Not sure it's appropriate for stage4 though; is this series fixing an important regression? Yes you

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread James Greenhalgh
On Tue, Mar 01, 2016 at 10:29:28AM +0100, Andreas Krebbel wrote: > On 02/29/2016 02:36 PM, Bernd Schmidt wrote: > > On 02/29/2016 09:46 AM, Andreas Krebbel wrote: > >> Ok for mainline? > >> > >>* gensupport.c (process_substs_on_one_elem): Split loop to > >>complete mark_operands_used_in_mat

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-03-01 Thread Andreas Krebbel
On 02/29/2016 02:36 PM, Bernd Schmidt wrote: > On 02/29/2016 09:46 AM, Andreas Krebbel wrote: >> Ok for mainline? >> >> * gensupport.c (process_substs_on_one_elem): Split loop to >> complete mark_operands_used_in_match_dup on all expressions in the >> vector first. >> (adjust_op

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-02-29 Thread Bernd Schmidt
On 02/29/2016 09:46 AM, Andreas Krebbel wrote: Ok for mainline? * gensupport.c (process_substs_on_one_elem): Split loop to complete mark_operands_used_in_match_dup on all expressions in the vector first. (adjust_operands_numbers): Inline into process_substs_on_one

Re: [PATCH 1/9] gensupport: Fix define_subst operand renumbering.

2016-01-18 Thread Bernd Schmidt
On 01/14/2016 05:33 PM, Andreas Krebbel wrote: When processing substitutions the operands are renumbered. To find a free operand number the array used_operands_numbers is used to record the operand numbers already in use. Currently this array is used to assign new numbers *before* all the RTXes