On Thu, 14 Mar 2019 at 15:35, Jakub Jelinek <ja...@redhat.com> wrote: > > On Thu, Mar 14, 2019 at 03:32:39PM +0100, Christophe Lyon wrote: > > > 2019-03-04 Richard Biener <rguent...@suse.de> > > > > > > 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 to backport this to gcc-8-branch when you committed > > r269650? > > That param is not supported in 8.x or older.
OK. I noticed this because I saw the same regressions I reported on trunk on some arm configs: https://gcc.gnu.org/ml/gcc-patches/2019-03/msg00125.html > > > > --- gcc/testsuite/gcc.dg/uninit-pred-8_b.c (revision 269361) > > > +++ gcc/testsuite/gcc.dg/uninit-pred-8_b.c (working copy) > > > @@ -1,6 +1,7 @@ > > > - > > > /* { dg-do compile } */ > > > -/* { dg-options "-Wuninitialized -O2" } */ > > > +/* ??? Jump threading makes a mess of the > > > logical-op-non-short-circuit=0 case > > > + so force it our way. */ > > > +/* { dg-options "-Wuninitialized -O2 --param > > > logical-op-non-short-circuit=1" } */ > > > > > > int g; > > > void bar(); > > Jakub