diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2006-10-10 11:46:59 +0200 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2006-10-10 11:46:59 +0200 |
| commit | 599a964a52716fadaa2e70337d36c54b9585cf0e (patch) | |
| tree | 81908999b4dc6913c4bb587a998b25cad6e17115 /gcc/builtins.c | |
| parent | be0436d7d964255fd5b9e69f3b2b00b6e301c3a6 (diff) | |
| download | gcc-599a964a52716fadaa2e70337d36c54b9585cf0e.zip gcc-599a964a52716fadaa2e70337d36c54b9585cf0e.tar.gz gcc-599a964a52716fadaa2e70337d36c54b9585cf0e.tar.bz2 | |
re PR middle-end/29272 (memcpy optimization causes wrong-code)
PR middle-end/29272
* builtins.c (var_decl_component_p): New function.
(fold_builtin_memset, fold_builtin_memory_op): Restrict
single entry optimization to variables and components thereof.
* gcc.c-torture/execute/20060930-2.c: New test.
From-SVN: r117599
Diffstat (limited to 'gcc/builtins.c')
| -rw-r--r-- | gcc/builtins.c | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/gcc/builtins.c b/gcc/builtins.c index 2997462..46f97f9 100644 --- a/gcc/builtins.c +++ b/gcc/builtins.c @@ -7893,6 +7893,17 @@ fold_builtin_exponent (tree fndecl, tree arglist, return 0; } +/* Return true if VAR is a VAR_DECL or a component thereof. */ + +static bool +var_decl_component_p (tree var) +{ + tree inner = var; + while (handled_component_p (inner)) + inner = TREE_OPERAND (inner, 0); + return SSA_VAR_P (inner); +} + /* Fold function call to builtin memset. Return NULL_TREE if no simplification can be made. */ @@ -7933,6 +7944,9 @@ fold_builtin_memset (tree arglist, tree type, bool ignore) && !POINTER_TYPE_P (TREE_TYPE (var))) return 0; + if (! var_decl_component_p (var)) + return 0; + length = tree_low_cst (len, 1); if (GET_MODE_SIZE (TYPE_MODE (TREE_TYPE (var))) != length || get_pointer_alignment (dest, BIGGEST_ALIGNMENT) / BITS_PER_UNIT @@ -8044,6 +8058,9 @@ fold_builtin_memory_op (tree arglist, tree type, bool ignore, int endp) && !SCALAR_FLOAT_TYPE_P (TREE_TYPE (destvar))) return 0; + if (! var_decl_component_p (destvar)) + return 0; + srcvar = src; STRIP_NOPS (srcvar); if (TREE_CODE (srcvar) != ADDR_EXPR) @@ -8058,6 +8075,9 @@ fold_builtin_memory_op (tree arglist, tree type, bool ignore, int endp) && !SCALAR_FLOAT_TYPE_P (TREE_TYPE (srcvar))) return 0; + if (! var_decl_component_p (srcvar)) + return 0; + length = tree_low_cst (len, 1); if (GET_MODE_SIZE (TYPE_MODE (TREE_TYPE (destvar))) != length || get_pointer_alignment (dest, BIGGEST_ALIGNMENT) / BITS_PER_UNIT |