From patchwork Mon Apr 18 11:36:51 2011
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: sel-sched: Fix erroneous re-use of pointer to last insn in a BB (PR
 48235)
From: Alexander Monakov <amonakov@ispras.ru>
X-Patchwork-Id: 91715
Message-Id: <alpine.LNX.2.00.1104181534000.17990@monoid.intra.ispras.ru>
To: Steve Ellcey <sje@cup.hp.com>
Cc: "Vladimir N. Makarov" <vmakarov@redhat.com>, Dmitry Melnik <dm@ispras.ru>, 
 gcc-patches@gcc.gnu.org
Date: Mon, 18 Apr 2011 15:36:51 +0400 (MSD)

On Fri, 15 Apr 2011, Steve Ellcey wrote:

> Vladimir and Dmitry,
> 
> The new test you added, gcc.dg/pr48235.c, is failing on IA64 because
> it gets this excess message:
> 
> cc1: note: -freorder-blocks-and-partition does not work on this architecture
> 
> Could you modify the test to either filter out this message or not 
> use the -freorder-blocks-and-partition option in IA64 or not run the
> test at all on IA64.  I notice that gcc.dg/tree-prof/pr45354.c looks
> like this test but also has:
> 
> /* { dg-require-effective-target freorder } */
> 
> That test does not fail, or run, on IA64 due to this dg option.

I have committed the following patch.  Sorry about the problem.

2011-04-18  Alexander Monakov  <amonakov@ispras.ru>

	* gcc.dg/pr48235.c: Add dg-require-effective-target freorder.


Index: gcc/testsuite/gcc.dg/pr48235.c
===================================================================
--- gcc/testsuite/gcc.dg/pr48235.c	(revision 172642)
+++ gcc/testsuite/gcc.dg/pr48235.c	(working copy)
@@ -1,4 +1,5 @@
 /* { dg-do compile { target powerpc*-*-* ia64-*-* i?86-*-* x86_64-*-* } } */
+/* { dg-require-effective-target freorder } */
 /* { dg-options "-O -fno-guess-branch-probability -fpeel-loops -freorder-blocks-and-partition -fschedule-insns2 -fsel-sched-pipelining -fselective-scheduling2" } */
 struct intC
 {