Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-14 Thread Richard Biener
On Thu, 14 Mar 2019, Christophe Lyon wrote: > On Thu, 14 Mar 2019 at 15:35, Jakub Jelinek wrote: > > > > On Thu, Mar 14, 2019 at 03:32:39PM +0100, Christophe Lyon wrote: > > > > 2019-03-04 Richard Biener > > > > > > > > PR testsuite/89551 > > > > * gcc.dg/uninit-pred-8_b.c: For

Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-14 Thread Christophe Lyon
On Thu, 14 Mar 2019 at 15:35, Jakub Jelinek wrote: > > On Thu, Mar 14, 2019 at 03:32:39PM +0100, Christophe Lyon wrote: > > > 2019-03-04 Richard Biener > > > > > > PR testsuite/89551 > > > * gcc.dg/uninit-pred-8_b.c: Force logical-op-non-short-circuit > > > the way that

Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-14 Thread Jakub Jelinek
On Thu, Mar 14, 2019 at 03:32:39PM +0100, Christophe Lyon wrote: > > 2019-03-04 Richard Biener > > > > PR testsuite/89551 > > * gcc.dg/uninit-pred-8_b.c: Force logical-op-non-short-circuit > > the way that makes the testcase PASS. > > Hi Richard, > > I think you forgot

Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-14 Thread Richard Biener
On March 14, 2019 3:32:39 PM GMT+01:00, Christophe Lyon wrote: >On Mon, 4 Mar 2019 at 11:25, Richard Biener wrote: >> >> >> The CFG cleanup change made us remove an extra forwarder which >somehow >> makes VRP jump threading go berzerk. Fortunately only on >> logical-op-non-short-circuit=0 targe

Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-14 Thread Christophe Lyon
On Mon, 4 Mar 2019 at 11:25, Richard Biener wrote: > > > The CFG cleanup change made us remove an extra forwarder which somehow > makes VRP jump threading go berzerk. Fortunately only on > logical-op-non-short-circuit=0 targets so the easy way to fix the > testcase is to force that our way. > > T

Re: [PATCH] Adjust gcc.dg/uninit-pred-8_b.c (PR89551)

2019-03-06 Thread Richard Biener
On Mon, 4 Mar 2019, Richard Biener wrote: > > The CFG cleanup change made us remove an extra forwarder which somehow > makes VRP jump threading go berzerk. Fortunately only on > logical-op-non-short-circuit=0 targets so the easy way to fix the > testcase is to force that our way. > > Tested on