diff options
| author | Richard Guenther <rguenther@suse.de> | 2007-11-09 12:32:20 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2007-11-09 12:32:20 +0000 |
| commit | 6aaa40131a37f28ffc3f1788b66293248b4abb5c (patch) | |
| tree | 7d2129cd384ed49e7ccc4d5da76366ff0ada7d49 /gcc/tree-ssa-ccp.c | |
| parent | 06acf7d0ad169b40c2e7f83c7ed808b2e196a6a4 (diff) | |
| download | gcc-6aaa40131a37f28ffc3f1788b66293248b4abb5c.zip gcc-6aaa40131a37f28ffc3f1788b66293248b4abb5c.tar.gz gcc-6aaa40131a37f28ffc3f1788b66293248b4abb5c.tar.bz2 | |
re PR target/33604 (Revision 119502 causes significantly slower results with 4.3/4.4 compared to 4.2)
2007-11-09 Richard Guenther <rguenther@suse.de>
PR tree-optimization/33604
* tree-ssa-forwprop.c (tree_ssa_forward_propagate_single_use_vars):
Disregard changes in CV qualifiers of pointed to types for
forward propagating ADDR_EXPRs.
* tree-ssa-ccp.c (fold_stmt_r): Preserve volatileness of the original
expression.
* g++.dg/tree-ssa/pr33604.C: New testcase.
* gcc.dg/pr32721.c: Adjust pattern.
From-SVN: r130040
Diffstat (limited to 'gcc/tree-ssa-ccp.c')
| -rw-r--r-- | gcc/tree-ssa-ccp.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/gcc/tree-ssa-ccp.c b/gcc/tree-ssa-ccp.c index 05a65f6..301316d 100644 --- a/gcc/tree-ssa-ccp.c +++ b/gcc/tree-ssa-ccp.c @@ -2034,6 +2034,7 @@ fold_stmt_r (tree *expr_p, int *walk_subtrees, void *data) bool *inside_addr_expr_p = fold_stmt_r_data->inside_addr_expr_p; bool *changed_p = fold_stmt_r_data->changed_p; tree expr = *expr_p, t; + bool volatile_p = TREE_THIS_VOLATILE (expr); /* ??? It'd be nice if walk_tree had a pre-order option. */ switch (TREE_CODE (expr)) @@ -2159,6 +2160,8 @@ fold_stmt_r (tree *expr_p, int *walk_subtrees, void *data) if (t) { + /* Preserve volatileness of the original expression. */ + TREE_THIS_VOLATILE (t) = volatile_p; *expr_p = t; *changed_p = true; } |