Patchwork sel-sched: Fix erroneous re-use of pointer to last insn in a BB (PR 48235)

login
register
mail settings
Submitter Alexander Monakov
Date April 18, 2011, 11:36 a.m.
Message ID <alpine.LNX.2.00.1104181534000.17990@monoid.intra.ispras.ru>
Download mbox | patch
Permalink /patch/91715/
State New
Headers show

Comments

Alexander Monakov - April 18, 2011, 11:36 a.m.
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.

Patch

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
 {