From: Matthias Klose <doko@gcc.gnu.org>
Date: Thu, 8 May 2014 22:17:43 +0000 (+0000)
Subject: re PR driver/61106 ([4.8/4.9] impliedness of -Wunused-parameter depends on -W option... 
X-Git-Tag: releases/gcc-5.1.0~7685
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=a82122df1d55d2e19e8d7442d952d13a4b3287f0;p=thirdparty%2Fgcc.git

re PR driver/61106 ([4.8/4.9] impliedness of -Wunused-parameter depends on -W option ordering)

gcc/

2014-05-08  Manuel López-Ibáñez  <manu@gcc.gnu.org>
            Matthias Klose  <doko@ubuntu.com>

        PR driver/61106
        * optc-gen.awk: Fix option handling for -Wunused-parameter.

gcc/testsuite/

2014-05-08  Matthias Klose  <doko@ubuntu.com>

        PR driver/61106
        * gcc-dg/unused-8a.c: New.
        * gcc-dg/unused-8b.c: Likewise.

From-SVN: r210246
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 89a88e2b5ba9..8e8b76e744df 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2014-05-08  Manuel López-Ibáñez  <manu@gcc.gnu.org>
+	    Matthias Klose  <doko@ubuntu.com>
+
+	PR driver/61106
+	* optc-gen.awk: Fix option handling for -Wunused-parameter.
+
 2014-05-08  Uros Bizjak  <ubizjak@gmail.com>
 
 	PR target/59952
diff --git a/gcc/optc-gen.awk b/gcc/optc-gen.awk
index a091b72af4ba..b70e4fe3aad2 100644
--- a/gcc/optc-gen.awk
+++ b/gcc/optc-gen.awk
@@ -406,11 +406,13 @@ for (i = 0; i < n_enabledby; i++) {
         if (opt_var_name != "") {
             condition = "!opts_set->x_" opt_var_name
             if (thisenableif[j] != "") {
-                condition = condition " && (" thisenableif[j] ")"
+                value = "(" thisenableif[j] ")"
+            } else {
+                value = "value"
             }
             print "      if (" condition ")"
             print "        handle_generated_option (opts, opts_set,"
-            print "                                 " opt_enum(thisenable[j]) ", NULL, value,"
+            print "                                 " opt_enum(thisenable[j]) ", NULL, " value ","
             print "                                 lang_mask, kind, loc, handlers, dc);"
         } else {
             print "#error " thisenable[j] " does not have a Var() flag"
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index bbfb333b90d6..2dcf9dcfe140 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2014-05-08  Matthias Klose  <doko@ubuntu.com>
+
+	PR driver/61106
+	* gcc-dg/unused-8a.c: New.
+	* gcc-dg/unused-8b.c: Likewise.
+
 2014-05-08  Marek Polacek  <polacek@redhat.com>
 
 	PR c/61053
diff --git a/gcc/testsuite/gcc.dg/unused-8a.c b/gcc/testsuite/gcc.dg/unused-8a.c
new file mode 100644
index 000000000000..b9b569051569
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/unused-8a.c
@@ -0,0 +1,4 @@
+/* { dg-do compile } */
+/* { dg-options "-Wall -Wextra -Wno-unused" } */
+
+void foo(int x) { }
diff --git a/gcc/testsuite/gcc.dg/unused-8b.c b/gcc/testsuite/gcc.dg/unused-8b.c
new file mode 100644
index 000000000000..5b4b894937cd
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/unused-8b.c
@@ -0,0 +1,4 @@
+/* { dg-do compile } */
+/* { dg-options "-Wall -Wno-unused -Wextra" } */
+
+void foo(int x) { }