Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 4 Jul 2007 23:01:48 +0400
From:      Andrey Chernov <ache@nagual.pp.ru>
To:        Ed Schouten <ed@fxq.nl>
Cc:        kan@freebsd.org, current@freebsd.org
Subject:   Re: [Patch] Silent gcc 4.2.0 loop optimization bug with -O2
Message-ID:  <20070704190148.GA34853@nagual.pp.ru>
In-Reply-To: <20070704121208.GC37187@hoeg.nl>
References:  <20070625130913.GA50273@nagual.pp.ru> <20070625133951.GA51324@nagual.pp.ru> <8e5ef5f70706250659v48f64410wb5399985a571087e@mail.gmail.com> <20070625145627.GA53685@nagual.pp.ru> <20070625151508.GO27942@hoeg.nl> <20070625152559.GA54055@nagual.pp.ru> <20070625153840.GP27942@hoeg.nl> <20070704121208.GC37187@hoeg.nl>

next in thread | previous in thread | raw e-mail | index | archive | help

--CE+1k2dSO48ffgeK
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 04, 2007 at 02:12:08PM +0200, Ed Schouten wrote:
> > Index: tree-ssa-loop-niter.c
> > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> > --- tree-ssa-loop-niter.c       (revision 126260)
> > +++ tree-ssa-loop-niter.c       (working copy)
> > @@ -1747,6 +1747,12 @@ infer_loop_bounds_from_undefined (struct
> >      {
> >        bb =3D bbs[i];
> >=20
> > +      /* If BB is not executed in each iteration of the loop, we cannot
> > +        use the operations in it to infer reliable upper bound on the
> > +        # of iterations of the loop.  */
> > +      if (!dominated_by_p (CDI_DOMINATORS, loop->latch, bb))
> > +       continue;
> > +
> >        for (bsi =3D bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi))
> >          {
> >           tree stmt =3D bsi_stmt (bsi);
> >=20
> > I'm going to test this.
> >=20
> ----- End forwarded message -----
>=20
> I just tested the patch on my desktop and it seems to work. The test
> code now compiles like it should. Hopefully it will be part of 4.2.1.

Will be nice if this patch will be commited in instead of my=20
sys.mk workaround. Alexander?

--=20
http://ache.pp.ru/

--CE+1k2dSO48ffgeK
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.4 (FreeBSD)

iD8DBQFGi+6cVg5YK5ZEdN0RAhzAAKCqE4wnrMMbAhPrp6Saz9tjcoU9nwCgoYpS
mlV2E7p++fOAz/AFl/Ri7pc=
=6bc6
-----END PGP SIGNATURE-----

--CE+1k2dSO48ffgeK--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070704190148.GA34853>