From 558af7ca1ccbaa309e390e871c620574a7446e4f Mon Sep 17 00:00:00 2001 From: Eric Botcazou Date: Wed, 1 Sep 2010 12:15:08 +0000 Subject: [PATCH] gimplify.c (gimplify_init_constructor): Do not create a temporary for a volatile LHS if... * gimplify.c (gimplify_init_constructor): Do not create a temporary for a volatile LHS if the constructor has only one element. From-SVN: r163727 --- gcc/ChangeLog | 17 +++++++++++------ gcc/gimplify.c | 7 ++++--- 2 files changed, 15 insertions(+), 9 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index ca67d7fc74f..fd03ae7f518 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,11 +1,16 @@ +2010-09-01 Eric Botcazou + + * gimplify.c (gimplify_init_constructor): Do not create a temporary for + a volatile LHS if the constructor has only one element. + 2010-09-01 Mikael Pettersson - PR bootstrap/45321 - * tree.c (stdarg_p): Make fntype parameter const. - * tree.h (stdarg_p): Likewise. - (function_args_iterator): Remove unused fntype field. - (function_args_iter_init): Do not initialize fntype - field. Make fntype parameter const. + PR bootstrap/45321 + * tree.c (stdarg_p): Make fntype parameter const. + * tree.h (stdarg_p): Likewise. + (function_args_iterator): Remove unused fntype field. + (function_args_iter_init): Do not initialize fntype + field. Make fntype parameter const. 2010-09-01 Richard Guenther diff --git a/gcc/gimplify.c b/gcc/gimplify.c index 52d83f656fb..b0468c6856b 100644 --- a/gcc/gimplify.c +++ b/gcc/gimplify.c @@ -3824,11 +3824,12 @@ gimplify_init_constructor (tree *expr_p, gimple_seq *pre_p, gimple_seq *post_p, } } - /* If the target is volatile and we have non-zero elements - initialize the target from a temporary. */ + /* If the target is volatile, we have non-zero elements and more than + one field to assign, initialize the target from a temporary. */ if (TREE_THIS_VOLATILE (object) && !TREE_ADDRESSABLE (type) - && num_nonzero_elements > 0) + && num_nonzero_elements > 0 + && VEC_length (constructor_elt, elts) > 1) { tree temp = create_tmp_var (TYPE_MAIN_VARIANT (type), NULL); TREE_OPERAND (*expr_p, 0) = temp;