diff options
| author | Richard Guenther <rguenther@suse.de> | 2008-09-16 12:34:01 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2008-09-16 12:34:01 +0000 |
| commit | d573123d17fb48ec675aa25f4e8a9754d876228c (patch) | |
| tree | 1dc33f94664f6b2d5b93ba40a31ffe948fc220aa /gcc/tree-sra.c | |
| parent | 9cb57027a39541712b049df895beda264d73d5ca (diff) | |
| download | gcc-d573123d17fb48ec675aa25f4e8a9754d876228c.zip gcc-d573123d17fb48ec675aa25f4e8a9754d876228c.tar.gz gcc-d573123d17fb48ec675aa25f4e8a9754d876228c.tar.bz2 | |
re PR middle-end/37380 (../../gcc/libcpp/charset.c:1103: error: 'cvt.77.width' is used uninitialized in this function)
2008-09-16 Richard Guenther <rguenther@suse.de>
PR middle-end/37380
* tree-sra.c (sra_build_assignment): Do not call the gimplifier
if not necessary.
* gcc.c-torture/compile/pr37380.c: New testcase.
From-SVN: r140388
Diffstat (limited to 'gcc/tree-sra.c')
| -rw-r--r-- | gcc/tree-sra.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/gcc/tree-sra.c b/gcc/tree-sra.c index 01f9084..3689b69 100644 --- a/gcc/tree-sra.c +++ b/gcc/tree-sra.c @@ -2308,8 +2308,14 @@ sra_build_assignment (tree dst, tree src) && !useless_type_conversion_p (TREE_TYPE (dst), TREE_TYPE (src))) src = fold_convert (TREE_TYPE (dst), src); - src = force_gimple_operand (src, &seq2, false, NULL_TREE); - gimple_seq_add_seq (&seq, seq2); + /* ??? Only call the gimplifier if we need to. Otherwise we may + end up substituting with DECL_VALUE_EXPR - see PR37380. */ + if (!handled_component_p (src) + && !SSA_VAR_P (src)) + { + src = force_gimple_operand (src, &seq2, false, NULL_TREE); + gimple_seq_add_seq (&seq, seq2); + } stmt = gimple_build_assign (dst, src); gimple_seq_add_stmt (&seq, stmt); return seq; |