This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: g++ 2.95.2 on powerpc
- To: Geoff Keating <geoffk at geoffk dot org>
- Subject: Re: g++ 2.95.2 on powerpc
- From: Franz Sirl <Franz dot Sirl-kernel at lauterbach dot com>
- Date: Mon, 13 Nov 2000 23:23:17 +0100
- Cc: gcc-patches at gcc dot gnu dot org
- References: <4vk8hlufrn.fsf@darbujan.fzu.cz> <00111321565300.02383@enzo.bigblue.local> <jmr94fzid5.fsf@geoffk.org>
On Monday 13 November 2000 22:12, Geoff Keating wrote:
> Franz Sirl <Franz.Sirl-kernel@lauterbach.com> writes:
> > On Thursday 27 April 2000 02:07, Geoff Keating wrote:
> > > Anyway, I will check in a modified version of the testcase as
> > > gcc.c-torture/execute/loop-5.c, since obviously this code doesn't get
> > > tickled enough.
> > >
> > > 2000-04-26 Geoff Keating <geoffk@cygnus.com>
> > >
> > > * loop.c (basic_induction_var): If a REG is set from something
> > > that is not a biv, then the REG is not a biv. Even if it is
> > > earlier set from something that is a biv.
> >
> > Geoff,
> >
> > Bernd noticed me that I accidentally committed this patch with my checkin
> > of
> >
> > 2000-09-29 Franz Sirl <Franz.Sirl-kernel@lauterbach.com>
> >
> > * loop.c (check_final_value): A GIV is not replaceable if used
> > before set.
> >
> > Any reason why your patch was never committed? Do we still need it? If
> > the patch is still OK, I'll just add your ChangeLog entry, otherwise I
> > will revert my accidental checkin.
>
> Since loop-5 seemed to pass before, I think the patch may not be
> needed. There was substantial work fixing things like this recently.
>
> If you happen to know that the patch does fix a bug, then leave it in;
> otherwise take it out.
Ok, I reverted it then. Sorry for my mistake and thanks to Bernd for noticing
it.
Franz.