diff options
| author | Richard Guenther <rguenther@suse.de> | 2011-10-10 15:45:53 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2011-10-10 15:45:53 +0000 |
| commit | a24ac4609a87536a62a27dd7976bf6ee0acc468f (patch) | |
| tree | f8a62b9ac946da08fd1b85696236a2dda0edcb5c /gcc | |
| parent | 40b6510f2e15672a4027cbd1395bb75ffd5920bc (diff) | |
| download | gcc-a24ac4609a87536a62a27dd7976bf6ee0acc468f.zip gcc-a24ac4609a87536a62a27dd7976bf6ee0acc468f.tar.gz gcc-a24ac4609a87536a62a27dd7976bf6ee0acc468f.tar.bz2 | |
re PR tree-optimization/50389 (ICE: in execute_todo, at passes.c:1730 with -O -freorder-blocks -ftracer and __builtin___memcpy_chk())
2011-10-10 Richard Guenther <rguenther@suse.de>
PR middle-end/50389
* gimple-fold.c (gimplify_and_update_call_from_tree): Do not
mark symbols for renaming. Append the VUSE to all statements
that possibly can have one.
* gcc.dg/torture/pr50389.c: New testcase.
From-SVN: r179757
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 7 | ||||
| -rw-r--r-- | gcc/gimple-fold.c | 11 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/torture/pr50389.c | 15 |
4 files changed, 31 insertions, 7 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 078c997..b3219b5 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,12 @@ 2011-10-10 Richard Guenther <rguenther@suse.de> + PR middle-end/50389 + * gimple-fold.c (gimplify_and_update_call_from_tree): Do not + mark symbols for renaming. Append the VUSE to all statements + that possibly can have one. + +2011-10-10 Richard Guenther <rguenther@suse.de> + * ipa-split.c (pass_split_functions): Add verification TODOs. (pass_feedback_split_functions): Likewise. diff --git a/gcc/gimple-fold.c b/gcc/gimple-fold.c index b4993fd..7df7e69 100644 --- a/gcc/gimple-fold.c +++ b/gcc/gimple-fold.c @@ -588,13 +588,10 @@ gimplify_and_update_call_from_tree (gimple_stmt_iterator *si_p, tree expr) } new_stmt = gsi_stmt (i); if (gimple_in_ssa_p (cfun)) - { - find_new_referenced_vars (new_stmt); - mark_symbols_for_renaming (new_stmt); - } - /* If the new statement has a VUSE, update it with exact SSA name we - know will reach this one. */ - if (gimple_vuse (new_stmt)) + find_new_referenced_vars (new_stmt); + /* If the new statement possibly has a VUSE, update it with exact SSA + name we know will reach this one. */ + if (gimple_has_mem_ops (new_stmt)) { /* If we've also seen a previous store create a new VDEF for the latter one, and make that the new reaching VUSE. */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 7d978b8..d3299d8 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,10 @@ 2011-10-10 Richard Guenther <rguenther@suse.de> + PR middle-end/50389 + * gcc.dg/torture/pr50389.c: New testcase. + +2011-10-10 Richard Guenther <rguenther@suse.de> + PR middle-end/50195 * gcc.dg/builtins-47.c: Optimize. diff --git a/gcc/testsuite/gcc.dg/torture/pr50389.c b/gcc/testsuite/gcc.dg/torture/pr50389.c new file mode 100644 index 0000000..6d8b095 --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/pr50389.c @@ -0,0 +1,15 @@ +/* { dg-do compile } */ +/* { dg-options "-freorder-blocks -ftracer" } */ + +extern int data[]; +extern int i; + +void +foo (void) +{ + char buf[8]; + __builtin___memcpy_chk (buf, data, i ? 8 : 4, + __builtin_object_size (buf, 0)); + __builtin___memcpy_chk (buf, data, i ? 8 : 4, + __builtin_object_size (buf, 0)); +} |