c-typeck.c (internal_build_compound_expr): Check for array to pointer conversion...

	* c-typeck.c (internal_build_compound_expr): Check for
	array to pointer conversion just once.

From-SVN: r41879

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c8f3f0aad44ea33e958b74f0eb3a8c8b37fe64a2..5d65f15d7b0aea4dda2c095cb82010b0be3d1f3c 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2001-05-06  Neil Booth  <neil@daikokuya.demon.co.uk>
+
+       * c-typeck.c (internal_build_compound_expr): Check for
+       array to pointer conversion just once.
+
 2001-05-05  Richard Henderson  <rth@redhat.com>
 
        * integrate.c (copy_insn_list): Substitute NOTE_EXPECTED_VALUE.

diff --git a/gcc/c-typeck.c b/gcc/c-typeck.c
index f05422ca6d21ee9b3f54b3447305264de3efc4e7..33ac1169a82956f87bf0c40231c051b38fc7da57 100644 (file)
--- a/gcc/c-typeck.c
+++ b/gcc/c-typeck.c
@@ -3644,6 +3644,10 @@ internal_build_compound_expr (list, first_p)
 
   if (TREE_CHAIN (list) == 0)
     {
+      /* Convert arrays to pointers when there really is a comma operator.  */
+      if (!first_p && TREE_CODE (TREE_TYPE (TREE_VALUE (list))) == ARRAY_TYPE)
+       TREE_VALUE (list) = default_conversion (TREE_VALUE (list));
+
 #if 0 /* If something inside inhibited lvalueness, we should not override.  */
       /* Consider (x, y+0), which is not an lvalue since y+0 is not.  */
 
@@ -3658,14 +3662,6 @@ internal_build_compound_expr (list, first_p)
       return TREE_VALUE (list);
     }
 
-  if (TREE_CHAIN (list) != 0 && TREE_CHAIN (TREE_CHAIN (list)) == 0)
-    {
-      /* Convert arrays to pointers when there really is a comma operator.  */
-      if (TREE_CODE (TREE_TYPE (TREE_VALUE (TREE_CHAIN (list)))) == ARRAY_TYPE)
-       TREE_VALUE (TREE_CHAIN (list))
-         = default_conversion (TREE_VALUE (TREE_CHAIN (list)));
-    }
-
   rest = internal_build_compound_expr (TREE_CHAIN (list), FALSE);
 
   if (! TREE_SIDE_EFFECTS (TREE_VALUE (list)))