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: [PR47036] sel-sched: don't attempt to move up unconditional jumps

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: Thu, 23 Dec 2010 11:57:33 -0500
Subject: Re: [PR47036] sel-sched: don't attempt to move up unconditional jumps
References: <alpine.LNX.2.00.1012222039540.5746@monoid.intra.ispras.ru>

On 12/22/2010 12:57 PM, Alexander Monakov wrote:

Hi,

This patch drops bits of support for moving up unconditional jumps.  The rest
of the scheduler does not handle it anyway, and most of the time dependencies
would prohibit such motion (in the testcase it's possible because other insns
are nops).

I've verified that the patch does not affect code generation on several cc1 .i
files at -O3 (targeting ia64).  Bootstrapped on x86-64-linux, ia64-linux
(testsuite running).  OK for trunk?

Ok. Thanks for fixing the bug.

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

	PR rtl-optimization/47036
	* sel-sched-ir.c (fallthru_bb_of_jump): Remove special support for
	unconditional jumps.
	* sel-sched.c (moveup_expr): Ditto.

testsuite:
	* g++.dg/opt/pr47036.C: New.

References:
- [PR47036] sel-sched: don't attempt to move up unconditional jumps
  - From: Alexander Monakov

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