From 8bf6e2702881e7d34ee9b127f716d30fc6ee0adf Mon Sep 17 00:00:00 2001 From: Revital Eres Date: Wed, 27 Feb 2008 13:27:56 +0000 Subject: [PATCH] Fix PR rtl-optimization/34999 From-SVN: r132711 --- gcc/ChangeLog | 9 +++++ gcc/bb-reorder.c | 48 +++++++++++++++++++----- gcc/testsuite/ChangeLog | 5 +++ gcc/testsuite/gcc.dg/tree-prof/pr34999.c | 45 ++++++++++++++++++++++ 4 files changed, 97 insertions(+), 10 deletions(-) create mode 100644 gcc/testsuite/gcc.dg/tree-prof/pr34999.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index aff90b89e5d..6c71d6fc2e3 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2008-02-27 Revital Eres + + PR rtl-optimization/34999 + * bb-reorder.c (add_labels_and_missing_jumps): Do not handle + crossing edges that ends with a call insn. + (fix_up_fall_thru_edges): Handle crossing edges that ends with a + call insn and clear the EDGE_CROSSING flag of the crossing edge + when fixing fallthru edges. + 2008-02-27 Richard Guenther PR middle-end/25290 diff --git a/gcc/bb-reorder.c b/gcc/bb-reorder.c index 352c9889172..18e9405a7ae 100644 --- a/gcc/bb-reorder.c +++ b/gcc/bb-reorder.c @@ -1289,11 +1289,14 @@ add_labels_and_missing_jumps (edge *crossing_edges, int n_crossing_edges) { label = block_label (dest); - /* Make sure source block ends with a jump. */ + /* Make sure source block ends with a jump. If the + source block does not end with a jump it might end + with a call_insn; this case will be handled in + fix_up_fall_thru_edges function. */ if (src && (src != ENTRY_BLOCK_PTR)) { - if (!JUMP_P (BB_END (src))) + if (!JUMP_P (BB_END (src)) && !block_ends_with_call_p (src)) /* bb just falls through. */ { /* make sure there's only one successor */ @@ -1318,13 +1321,13 @@ add_labels_and_missing_jumps (edge *crossing_edges, int n_crossing_edges) } /* Find any bb's where the fall-through edge is a crossing edge (note that - these bb's must also contain a conditional jump; we've already - dealt with fall-through edges for blocks that didn't have a - conditional jump in the call to add_labels_and_missing_jumps). - Convert the fall-through edge to non-crossing edge by inserting a - new bb to fall-through into. The new bb will contain an - unconditional jump (crossing edge) to the original fall through - destination. */ + these bb's must also contain a conditional jump or end with a call + instruction; we've already dealt with fall-through edges for blocks + that didn't have a conditional jump or didn't end with call instruction + in the call to add_labels_and_missing_jumps). Convert the fall-through + edge to non-crossing edge by inserting a new bb to fall-through into. + The new bb will contain an unconditional jump (crossing edge) to the + original fall through destination. */ static void fix_up_fall_thru_edges (void) @@ -1369,6 +1372,19 @@ fix_up_fall_thru_edges (void) fall_thru = succ2; cond_jump = succ1; } + else if (!fall_thru && succ1 && block_ends_with_call_p (cur_bb)) + { + edge e; + edge_iterator ei; + + /* Find EDGE_CAN_FALLTHRU edge. */ + FOR_EACH_EDGE (e, ei, cur_bb->succs) + if (e->flags & EDGE_CAN_FALLTHRU) + { + fall_thru = e; + break; + } + } if (fall_thru && (fall_thru->dest != EXIT_BLOCK_PTR)) { @@ -1426,8 +1442,14 @@ fix_up_fall_thru_edges (void) /* This is the case where both edges out of the basic block are crossing edges. Here we will fix up the fall through edge. The jump edge will be taken care - of later. */ + of later. The EDGE_CROSSING flag of fall_thru edge + is unset before the call to force_nonfallthru + function because if a new basic-block is created + this edge remains in the current section boundary + while the edge between new_bb and the fall_thru->dest + becomes EDGE_CROSSING. */ + fall_thru->flags &= ~EDGE_CROSSING; new_bb = force_nonfallthru (fall_thru); if (new_bb) @@ -1441,6 +1463,12 @@ fix_up_fall_thru_edges (void) BB_COPY_PARTITION (new_bb, cur_bb); single_succ_edge (new_bb)->flags |= EDGE_CROSSING; } + else + { + /* If a new basic-block was not created; restore + the EDGE_CROSSING flag. */ + fall_thru->flags |= EDGE_CROSSING; + } /* Add barrier after new jump */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 51196bb03ae..5de4552a107 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2008-02-27 Revital Eres + + PR rtl-optimization/34999 + * gcc.dg/tree-prof/pr34999.c: New. + 2008-02-27 Richard Guenther PR middle-end/25290 diff --git a/gcc/testsuite/gcc.dg/tree-prof/pr34999.c b/gcc/testsuite/gcc.dg/tree-prof/pr34999.c new file mode 100644 index 00000000000..4ec42790e57 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-prof/pr34999.c @@ -0,0 +1,45 @@ +/* Same test as built-in-setjmp.c. Includes the case where + the source block of a crossing fallthru edge ends with a call. */ +/* { dg-require-effective-target freorder } */ +/* { dg-options "-O2 -freorder-blocks-and-partition" } */ + +extern int strcmp(const char *, const char *); +extern char *strcpy(char *, const char *); +extern void abort(void); +extern void exit(int); + +void *buf[20]; + +void __attribute__((noinline)) +sub2 (void) +{ + __builtin_longjmp (buf, 1); +} + +int +main () +{ + char *p = (char *) __builtin_alloca (20); + + strcpy (p, "test"); + + if (__builtin_setjmp (buf)) + { + if (strcmp (p, "test") != 0) + abort (); + + exit (0); + } + + { + int *q = (int *) __builtin_alloca (p[2] * sizeof (int)); + int i; + + for (i = 0; i < p[2]; i++) + q[i] = 0; + + while (1) + sub2 (); + } +} +