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

Reply via email to