* gimplify.c (gimplify_init_ctor_eval): Don't discard a zero-sized

	value if it has side-effects.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@104262 138bc75d-0d04-0410-961f-82ee72b054a4

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index d4944d5..a0da848 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2005-09-14  Olivier Hainque  <hainque@adacore.com>
+
+       * gimplify.c (gimplify_init_ctor_eval): Don't discard a zero-sized
+       value if it has side-effects.
+
 2005-09-14  David Edelsohn  <edelsohn@gnu.org>
 
        PR target/22068

diff --git a/gcc/gimplify.c b/gcc/gimplify.c
index a2fc7c8..cdb8a33 100644 (file)
--- a/gcc/gimplify.c
+++ b/gcc/gimplify.c
@@ -2576,7 +2576,12 @@ gimplify_init_ctor_eval (tree object, VEC(constructor_elt,gc) *elts,
         so we don't have to figure out what's missing ourselves.  */
       gcc_assert (purpose);
 
-      if (zero_sized_field_decl (purpose))
+      /* Skip zero-sized fields, unless value has side-effects.  This can
+        happen with calls to functions returning a zero-sized type, which
+        we shouldn't discard.  As a number of downstream passes don't
+        expect sets of zero-sized fields, we rely on the gimplification of
+        the MODIFY_EXPR we make below to drop the assignment statement.  */
+      if (! TREE_SIDE_EFFECTS (value) && zero_sized_field_decl (purpose))
        continue;
 
       /* If we have a RANGE_EXPR, we have to build a loop to assign the