[PATCH] Simplify rest_of_compilation logic

Matt Kraai kraai@alumni.cmu.edu
Fri Jun 14 06:36:00 GMT 2002


On Thu, Jun 13, 2002 at 11:53:39AM -0700, Matt Kraai wrote:
> On Thu, Jun 13, 2002 at 12:45:22PM -0600, law@redhat.com wrote:
> > In message <20020613183935.GB4837@catalonia>, Matt Kraai writes:
> >  > 
> >  > Content-Type: text/plain; charset=us-ascii
> >  > Content-Disposition: inline
> >  > Content-Transfer-Encoding: quoted-printable
> >  > 
> >  > On Thu, Jun 13, 2002 at 10:03:27AM -0600, law@redhat.com wrote:
> >  > >  In message <Pine.LNX.4.33.0206130734590.11151-100000@www.eyesopen.com>, =
> >  > Roger=20
> >  > > Sayle writes:
> >  > >  > A potential further improvement still would be to have the function
> >  > >  > delete_null_pointer_checks return a bool indicating that it actually
> >  > >  > modified one or more jumps, and use this in rest_of_compilation to
> >  > >  > determine whether it needs to call cleanup_cfg afterwards.
> >  > > Yes.  I actually thought the code did this, but looking at it now, it
> >  > > seems to not be the case.  It should be relatively simple to propagate
> >  > > out a return value to indicate if something was changed.
> >  > 
> >  > The following patch does so.  It compiles, but I cannot
> >  > bootstrap or regression test it since bootstrapping on PowerPC
> >  > is currently broken.
> 
> Of course, I accidentally hosed the patch, so I regenerated it
> by hand, which of course didn't work.  Argh.  Fixed version
> appended.

I bootstrapped and regression tested the patch[1] by reverting
Jeff's changes to gen_rtx_REG.  OK to commit?

Matt

1. http://gcc.gnu.org/ml/gcc-patches/2002-06/msg01206.html
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 232 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20020614/a328b383/attachment.sig>


More information about the Gcc-patches mailing list