From: Jakub Jelinek <jakub@redhat.com>
Date: Tue, 4 May 2010 19:08:37 +0000 (+0200)
Subject: re PR c/43981 (-Wunused-but-set-variable does not consider array sizing use of a... 
X-Git-Tag: releases/gcc-4.6.0~7497
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=97e3c92355da5e885866e65dd1c1c6903f8611c2;p=thirdparty%2Fgcc.git

re PR c/43981 (-Wunused-but-set-variable does not consider array sizing use of a variable)

	PR c/43981
	* c-parser.c (c_parser_direct_declarator_inner): Call mark_exp_read
	on dimen.

	* gcc.dg/Wunused-var-8.c: New test.

From-SVN: r159044
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 61797432b8d4..21f145e9bc22 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2010-05-04  Jakub Jelinek  <jakub@redhat.com>
+
+	PR c/43981
+	* c-parser.c (c_parser_direct_declarator_inner): Call mark_exp_read
+	on dimen.
+
 2010-05-04  H.J. Lu  <hongjiu.lu@intel.com>
 
 	PR target/43799
diff --git a/gcc/c-parser.c b/gcc/c-parser.c
index cbdee1eb8a14..e5440d47caf1 100644
--- a/gcc/c-parser.c
+++ b/gcc/c-parser.c
@@ -2474,6 +2474,8 @@ c_parser_direct_declarator_inner (c_parser *parser, bool id_present,
 				     "expected %<]%>");
 	  return NULL;
 	}
+      if (dimen)
+	mark_exp_read (dimen);
       declarator = build_array_declarator (brace_loc, dimen, quals_attrs,
 					   static_seen, star_seen);
       if (declarator == NULL)
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 5b213ca82fda..4b2fe7d57adc 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2010-05-04  Jakub Jelinek  <jakub@redhat.com>
+
+	PR c/43981
+	* gcc.dg/Wunused-var-8.c: New test.
+
 2010-05-04  Jason Merrill  <jason@redhat.com>
 
 	PR c++/38064
diff --git a/gcc/testsuite/gcc.dg/Wunused-var-8.c b/gcc/testsuite/gcc.dg/Wunused-var-8.c
new file mode 100644
index 000000000000..1217719a2988
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/Wunused-var-8.c
@@ -0,0 +1,15 @@
+/* PR c/43981 */
+/* { dg-do compile } */
+/* { dg-options "-Wunused" } */
+
+void g (char *);
+
+char
+f (int a)
+{
+  int len = a * 3;
+  char t[len];
+
+  g (t);
+  return t[0];
+}