diff options
author | Richard Guenther <rguenther@suse.de> | 2008-01-04 15:56:41 +0000 |
---|---|---|
committer | Richard Biener <rguenth@gcc.gnu.org> | 2008-01-04 15:56:41 +0000 |
commit | bdb69beee91adebad0d3b6b0d3dd0f3810dbe7a2 (patch) | |
tree | b5fe9fbbe6e89d6fae4bc8453d5d165723a585b6 /gcc | |
parent | 578c21cb86c58d3b6e917b3943b78a9d06f54f56 (diff) | |
download | gcc-bdb69beee91adebad0d3b6b0d3dd0f3810dbe7a2.zip gcc-bdb69beee91adebad0d3b6b0d3dd0f3810dbe7a2.tar.gz gcc-bdb69beee91adebad0d3b6b0d3dd0f3810dbe7a2.tar.bz2 |
re PR tree-optimization/34029 (internal compiler error: verify_stmts failed)
2008-01-04 Richard Guenther <rguenther@suse.de>
PR middle-end/34029
* tree-cfg.c (verify_expr): Do not look inside ADDR_EXPRs
for verifying purposes if they are is_gimple_min_invariant.
* gcc.c-torture/compile/pr34029-1.c: New testcase.
* gcc.c-torture/compile/pr34029-2.c: Likewise.
From-SVN: r131325
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 6 | ||||
-rw-r--r-- | gcc/testsuite/gcc.c-torture/compile/pr34029-1.c | 22 | ||||
-rw-r--r-- | gcc/testsuite/gcc.c-torture/compile/pr34029-2.c | 12 | ||||
-rw-r--r-- | gcc/tree-cfg.c | 5 |
5 files changed, 51 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d046801..d16ff84 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2008-01-04 Richard Guenther <rguenther@suse.de> + + PR middle-end/34029 + * tree-cfg.c (verify_expr): Do not look inside ADDR_EXPRs + for verifying purposes if they are is_gimple_min_invariant. + 2008-01-04 Aldy Hernandez <aldyh@redhat.com> PR tree-optimization/34448 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 239fbeb..a8cdbf7 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2008-01-04 Richard Guenther <rguenther@suse.de> + + PR middle-end/34029 + * gcc.c-torture/compile/pr34029-1.c: New testcase. + * gcc.c-torture/compile/pr34029-2.c: Likewise. + 2008-01-04 Jakub Jelinek <jakub@redhat.com> PR gcov-profile/34609 diff --git a/gcc/testsuite/gcc.c-torture/compile/pr34029-1.c b/gcc/testsuite/gcc.c-torture/compile/pr34029-1.c new file mode 100644 index 0000000..5261129 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr34029-1.c @@ -0,0 +1,22 @@ +static const char s[] = "ab.cd.efghijk"; + +int +foo (const char *x) +{ + const char *a; + int b = 0; + + a = __builtin_strchr (s, '.'); + if (a == 0) + b = 1; + else if ((a = __builtin_strchr (a + 1, '.')) == 0) + b = 1; + else if (__builtin_strncmp (s, x, a - s)) + b = 1; + else if (__builtin_strncmp (a + 1, x + (a - s + 1), 4) < 0) + b = 1; + + if (b) + return 4; + return 0; +} diff --git a/gcc/testsuite/gcc.c-torture/compile/pr34029-2.c b/gcc/testsuite/gcc.c-torture/compile/pr34029-2.c new file mode 100644 index 0000000..9ec49ec --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr34029-2.c @@ -0,0 +1,12 @@ +static const char s[] = "ab.cd.efghijk"; +static const char t[] = "abcde"; + +long +foo (const char *x) +{ + const char *a; + long b = 0; + + a = __builtin_strchr (s, '.'); + return ((long) a) + (1 - (long) t); +} diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index 1606497..a8cb627 100644 --- a/gcc/tree-cfg.c +++ b/gcc/tree-cfg.c @@ -3218,6 +3218,11 @@ verify_expr (tree *tp, int *walk_subtrees, void *data ATTRIBUTE_UNUSED) error ("address taken, but ADDRESSABLE bit not set"); return x; } + + /* Stop recursing and verifying invariant ADDR_EXPRs, they tend + to become arbitrary complicated. */ + if (is_gimple_min_invariant (t)) + *walk_subtrees = 0; break; } |