Re: [PATCH] Fix endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334, take 2)

2018-02-15 Thread Richard Biener
On Wed, 14 Feb 2018, Jakub Jelinek wrote: > On Tue, Feb 13, 2018 at 07:04:09PM +0100, Richard Biener wrote: > > On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek > > wrote: > > >On the following testcase, we recurse infinitely, because > > >we have float re-association

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-14 Thread Jakub Jelinek
On Wed, Feb 14, 2018 at 12:09:57PM +0100, Richard Biener wrote: > On Tue, 13 Feb 2018, Marc Glisse wrote: > > > On Tue, 13 Feb 2018, Richard Biener wrote: > > > > > On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek > > > > > > wrote: > > > > Hi! > > > > > > > > On the

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-14 Thread Marc Glisse
On Wed, 14 Feb 2018, Richard Biener wrote: On Tue, 13 Feb 2018, Marc Glisse wrote: On Tue, 13 Feb 2018, Richard Biener wrote: On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek wrote: Hi! On the following testcase, we recurse infinitely, because we have float

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-14 Thread Richard Biener
On Tue, 13 Feb 2018, Marc Glisse wrote: > On Tue, 13 Feb 2018, Richard Biener wrote: > > > On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek > > wrote: > > > Hi! > > > > > > On the following testcase, we recurse infinitely, because > > > we have float re-association

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-13 Thread Jakub Jelinek
On Tue, Feb 13, 2018 at 03:28:25PM -0400, Marc Glisse wrote: > On Tue, 13 Feb 2018, Richard Biener wrote: > > > On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek > > wrote: > > > Hi! > > > > > > On the following testcase, we recurse infinitely, because > > > we have

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-13 Thread Marc Glisse
On Tue, 13 Feb 2018, Richard Biener wrote: On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek wrote: Hi! On the following testcase, we recurse infinitely, because we have float re-association enabled, but also rounding-math, so we try to optimize (cst1 + cst2) + cst3

Re: [PATCH] FIx endless match.pd recursion on cst1 + cst2 + cst3 (PR tree-optimization/84334)

2018-02-13 Thread Richard Biener
On February 13, 2018 6:51:29 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >On the following testcase, we recurse infinitely, because >we have float re-association enabled, but also rounding-math, so >we try to optimize (cst1 + cst2) + cst3 as (cst2 + cst3) + cst1 >but (cst2 +