This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]

Re: PATCH: small -freorder-block defect

To: Richard Henderson <rth at cygnus dot com>
Subject: Re: PATCH: small -freorder-block defect
From: Franz Sirl <Franz dot Sirl-kernel at lauterbach dot com>
Date: Wed, 15 Mar 2000 01:50:30 +0100
Cc: Jason Lee Eckhardt <jle at owlnet dot rice dot edu>,gcc-patches at gcc dot gnu dot org
References: <4.3.2.20000315003013.00c3cd70@mail.lauterbach.com><Pine.GSO.4.21.0003141531310.23559-100000@forest.owlnet.ric e.edu><4.3.2.20000315003013.00c3cd70@mail.lauterbach.com>

At 01:34 15.03.00, Richard Henderson wrote:
>On Wed, Mar 15, 2000 at 12:37:09AM +0100, Franz Sirl wrote:
> > This causes a bootstrap fail on powerpc-linux-gnu with newppc-branch (and
> > probably other HAVE_return targets.
> >
> > The appended patch seems to fix it for me. Opinions?
>
>Ah.  This means Jason's patch is wrong.  A conditional return
>should not be followed by a barrier.

Heh, that's actually good because my patch doesn't work, gcc will happily 
remove BBs after a conditional return with a barrier following, breaking 
the bootstrap lateron.

BTW, the dumpfilename for BBRO misses the numbering, probably it should be 
".165.bbro" or similar.

Franz.

References:
- Re: PATCH: small -freorder-block defect
  - From: Franz Sirl
- Re: PATCH: small -freorder-block defect
  - From: Richard Henderson

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]