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]
Other format:	[Raw text]

Re: [PR46649] sel-sched: don't try to delete the first BB in the region

From: Vladimir Makarov <vmakarov at redhat dot com>
To: Alexander Monakov <amonakov at ispras dot ru>
Cc: gcc-patches at gcc dot gnu dot org
Date: Tue, 14 Dec 2010 12:35:24 -0500
Subject: Re: [PR46649] sel-sched: don't try to delete the first BB in the region
References: <alpine.LNX.2.00.1012141853520.5687@monoid.intra.ispras.ru>

On 12/14/2010 12:29 PM, Alexander Monakov wrote:

Hi,

Our BB removal machinery can not delete the first BB in the region.  This
patch restores the correct behaviour for purge_empty_blocks: we should skip
the first BB unconditionally, not only if it's a preheader.  It was
accidentally broken a while ago while fixing a different bug because I was not
aware of the limitation.

Bootstrapped& tested on x86_64-linux, OK for trunk?

Ok, thanks.

2010-12-14 Alexander Monakov<amonakov@ispras.ru>

	PR rtl-optimization/46649
	* sel-sched-ir.c (purge_empty_blocks): Unconditionally skip the first
	basic block in the region.

testsuite:
	g++.dg/opt/pr46649.C: New.

References:
- [PR46649] sel-sched: don't try to delete the first BB in the region
  - From: Alexander Monakov

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