Re: fwprop fix for PR79405

2017-02-24 Thread Richard Biener
On Thu, Feb 23, 2017 at 11:17 PM, Jeff Law wrote: > On 02/23/2017 01:57 AM, Richard Biener wrote: >> >> On Wed, Feb 22, 2017 at 6:19 PM, Jeff Law wrote: >>> >>> On 02/16/2017 12:41 PM, Bernd Schmidt wrote: We have two registers being assigned to

Re: fwprop fix for PR79405

2017-02-23 Thread Jeff Law
On 02/23/2017 01:57 AM, Richard Biener wrote: On Wed, Feb 22, 2017 at 6:19 PM, Jeff Law wrote: On 02/16/2017 12:41 PM, Bernd Schmidt wrote: We have two registers being assigned to each other: (set (reg 213) (reg 209)) (set (reg 209) (reg 213)) These being the only

Re: fwprop fix for PR79405

2017-02-23 Thread Richard Biener
On Wed, Feb 22, 2017 at 6:19 PM, Jeff Law wrote: > On 02/16/2017 12:41 PM, Bernd Schmidt wrote: >> >> We have two registers being assigned to each other: >> >> (set (reg 213) (reg 209)) >> (set (reg 209) (reg 213)) >> >> These being the only definitions, we are happy to forward

Re: fwprop fix for PR79405

2017-02-22 Thread Jeff Law
On 02/16/2017 12:41 PM, Bernd Schmidt wrote: We have two registers being assigned to each other: (set (reg 213) (reg 209)) (set (reg 209) (reg 213)) These being the only definitions, we are happy to forward propagate reg 209 for reg 213 into a third insn, making a new use for reg 209. We are

Re: fwprop fix for PR79405

2017-02-20 Thread Richard Biener
On February 20, 2017 2:58:54 PM GMT+01:00, Bernd Schmidt wrote: >On 02/17/2017 10:11 AM, Richard Biener wrote: >> Index: gcc/fwprop.c >> === >> --- gcc/fwprop.c(revision 245501) >> +++ gcc/fwprop.c

Re: fwprop fix for PR79405

2017-02-20 Thread Bernd Schmidt
On 02/17/2017 10:11 AM, Richard Biener wrote: Index: gcc/fwprop.c === --- gcc/fwprop.c(revision 245501) +++ gcc/fwprop.c(working copy) @@ -1478,7 +1478,8 @@ fwprop (void) Do not forward propagate addresses into

Re: fwprop fix for PR79405

2017-02-17 Thread Richard Biener
On Fri, Feb 17, 2017 at 10:07 AM, Richard Biener wrote: > On Thu, Feb 16, 2017 at 8:41 PM, Bernd Schmidt wrote: >> We have two registers being assigned to each other: >> >> (set (reg 213) (reg 209)) >> (set (reg 209) (reg 213)) >> >> These being

Re: fwprop fix for PR79405

2017-02-17 Thread Richard Biener
On Thu, Feb 16, 2017 at 8:41 PM, Bernd Schmidt wrote: > We have two registers being assigned to each other: > > (set (reg 213) (reg 209)) > (set (reg 209) (reg 213)) > > These being the only definitions, we are happy to forward propagate reg 209 > for reg 213 into a third

fwprop fix for PR79405

2017-02-16 Thread Bernd Schmidt
We have two registers being assigned to each other: (set (reg 213) (reg 209)) (set (reg 209) (reg 213)) These being the only definitions, we are happy to forward propagate reg 209 for reg 213 into a third insn, making a new use for reg 209. We are then happy to forward propagate reg 213 for