diff options
| author | Diego Novillo <dnovillo@redhat.com> | 2005-11-15 02:59:03 +0000 |
|---|---|---|
| committer | Diego Novillo <dnovillo@gcc.gnu.org> | 2005-11-14 21:59:03 -0500 |
| commit | 8c5285e1cbc832dee329364e1c4690a7d28a313f (patch) | |
| tree | 01472ccbe2e961644ad8b9fcf25f37493140d337 /gcc | |
| parent | 1f4c2c57da3143ef0a5e9943c8e16ab05a0883d9 (diff) | |
| download | gcc-8c5285e1cbc832dee329364e1c4690a7d28a313f.zip gcc-8c5285e1cbc832dee329364e1c4690a7d28a313f.tar.gz gcc-8c5285e1cbc832dee329364e1c4690a7d28a313f.tar.bz2 | |
re PR tree-optimization/24840 (ICE process_assert_insertions_for, at tree-vrp.c:2807)
PR 24840
* tree-vrp.c (infer_value_range): Return false if STMT is a
block terminator and its basic block has no successors.
testsuite/
PR 24840
* gcc.dg/tree-ssa/pr24840.c: New test.
From-SVN: r106930
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/tree-ssa/pr24840.c | 17 | ||||
| -rw-r--r-- | gcc/tree-vrp.c | 7 |
4 files changed, 35 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 032b129..804e6b5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-11-14 Diego Novillo <dnovillo@redhat.com> + + PR 24840 + * tree-vrp.c (infer_value_range): Return false if STMT is a + block terminator and its basic block has no successors. + 2005-11-14 Mike Stump <mrs@apple.com> * config/i386/i386.c (override_options): -masm=intel isn't diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index ba2ffa2..2c2f397 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-11-14 Diego Novillo <dnovillo@redhat.com> + + PR 24840 + * gcc.dg/tree-ssa/pr24840.c: New test. + 2005-11-14 Janis Johnson <janis187@us.ibm.com> PR testsuite/24841 diff --git a/gcc/testsuite/gcc.dg/tree-ssa/pr24840.c b/gcc/testsuite/gcc.dg/tree-ssa/pr24840.c new file mode 100644 index 0000000..736dafd --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/pr24840.c @@ -0,0 +1,17 @@ +/* { dg-do compile } */ +/* { dg-options "-O2" } */ + +typedef struct +{ + int tag, y; +} Object; + +void Err_Handler () __attribute__ ((__noreturn__)); + +int +P_Error (int argc, Object * argv) +{ + if (((argv[1]).tag >> 1) != 11) + Wrong_Type (argv[1], 11); + Err_Handler (argv[0], argv[1], argc - 2, argv + 2); +} diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index d49c4d4..f88b23e 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -2221,6 +2221,13 @@ infer_value_range (tree stmt, tree op, enum tree_code *comp_code_p, tree *val_p) if (tree_could_throw_p (stmt)) return false; + /* If STMT is the last statement of a basic block with no + successors, there is no point inferring anything about any of its + operands. We would not be able to find a proper insertion point + for the assertion, anyway. */ + if (stmt_ends_bb_p (stmt) && EDGE_COUNT (bb_for_stmt (stmt)->succs) == 0) + return false; + if (POINTER_TYPE_P (TREE_TYPE (op))) { bool is_store; |