diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2010-05-21 20:45:29 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2010-05-21 20:45:29 +0200 |
| commit | 14db98d4a6349271ab34e894c9b0780cb985666b (patch) | |
| tree | ecc6ff1833b54604393846a37e39a3473d3c215c | |
| parent | d49b6e1e33707adeb972ade7cec09acc0b178d02 (diff) | |
| download | gcc-14db98d4a6349271ab34e894c9b0780cb985666b.zip gcc-14db98d4a6349271ab34e894c9b0780cb985666b.tar.gz gcc-14db98d4a6349271ab34e894c9b0780cb985666b.tar.bz2 | |
re PR debug/44223 (segmentation fault with -g -fsched-pressure)
PR debug/44223
* haifa-sched.c (schedule_insn): When freeing INSN_REG_USE_LIST,
unchain each use from the cyclic next_regno_use chain first.
* gcc.target/i386/pr44223.c: New test.
From-SVN: r159680
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/haifa-sched.c | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.target/i386/pr44223.c | 36 |
4 files changed, 53 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ee78f23..dcfff77 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-05-21 Jakub Jelinek <jakub@redhat.com> + + PR debug/44223 + * haifa-sched.c (schedule_insn): When freeing INSN_REG_USE_LIST, + unchain each use from the cyclic next_regno_use chain first. + 2010-05-21 Steven Bosscher <steven@gcc.gnu.org> * real: Do not include gmp.h, mpfr.h, and mpc.h. diff --git a/gcc/haifa-sched.c b/gcc/haifa-sched.c index f7c087f..b15fe63 100644 --- a/gcc/haifa-sched.c +++ b/gcc/haifa-sched.c @@ -1721,6 +1721,12 @@ schedule_insn (rtx insn) /* Unknown location doesn't use any registers. */ for (use = INSN_REG_USE_LIST (dbg); use != NULL; use = next) { + struct reg_use_data *prev = use; + + /* Remove use from the cyclic next_regno_use chain first. */ + while (prev->next_regno_use != use) + prev = prev->next_regno_use; + prev->next_regno_use = use->next_regno_use; next = use->next_insn_use; free (use); } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 259ee77..d40fc04 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-05-21 Jakub Jelinek <jakub@redhat.com> + + PR debug/44223 + * gcc.target/i386/pr44223.c: New test. + 2010-05-21 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> * gcc.target/i386/pr25993.c: Use @function as type specifier. diff --git a/gcc/testsuite/gcc.target/i386/pr44223.c b/gcc/testsuite/gcc.target/i386/pr44223.c new file mode 100644 index 0000000..3b8030c --- /dev/null +++ b/gcc/testsuite/gcc.target/i386/pr44223.c @@ -0,0 +1,36 @@ +/* PR debug/44223 */ +/* { dg-do compile } */ +/* { dg-options "-O3 -fsched-pressure -fschedule-insns -fpic -march=core2 -g" { target fpic } } */ + +struct S { unsigned int s1; int s2; }; +struct T { int t; }; + +extern void extfn (struct S *); + +static inline void +foo (struct S *s, unsigned char *x, int y) +{ + s->s2 = 32; +} + +static inline void +bar (struct S *s, int n, unsigned int x) +{ + unsigned int s1; + int s2; + s1 = s->s1; + s2 = s->s2; + if (n < s2) + s1 = (s1 << n) | x; + s->s1 = s1; +} + +int +baz (struct T *u, unsigned char *v, int w) +{ + struct S y; + foo (&y, v, 7); + bar (&y, 12, 0xfff); + bar (&y, 2, u->t); + extfn (&y); +} |