re PR c/67819 (-Wduplicated-cond should take macros into account)

	* genemit.c (gen_exp): Remove -Wduplicated-cond hack.

	* c.opt (Wduplicated-cond): Don't enable by -Wall anymore.

	* c-c++-common/Wduplicated-cond-2.c: Skip until PR67819 is resolved.

From-SVN: r228405

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index f9e53727168744ddd116eecc6b0244451ae71361..6e4181092fef40e60e7d2697d2e233f54075da7e 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2015-10-02  Marek Polacek  <polacek@redhat.com>
+
+       * genemit.c (gen_exp): Remove -Wduplicated-cond hack.
+
 2015-10-02  Aditya Kumar  <aditya.k7@samsung.com>
 
        * graphite-scop-detection.c (loop_ivs_can_be_represented): New.

diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog
index 8ab8964a8c535c5073a1dda44ce1c9f52ab8790f..78cc2bdc8b83172de16c75c363ea02664f30e6e1 100644 (file)
--- a/gcc/c-family/ChangeLog
+++ b/gcc/c-family/ChangeLog
@@ -1,3 +1,7 @@
+2015-10-02  Marek Polacek  <polacek@redhat.com>
+
+       * c.opt (Wduplicated-cond): Don't enable by -Wall anymore.
+
 2015-10-02  Marek Polacek  <polacek@redhat.com>
 
        PR c/64249

diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
index a79b9f1a39f90d4684db3061c87a4ca66f0714f3..c62eefb98c28a25d9d6ce67c2cb79673cf299910 100644 (file)
--- a/gcc/c-family/c.opt
+++ b/gcc/c-family/c.opt
@@ -407,7 +407,7 @@ C ObjC C++ ObjC++ Var(warn_div_by_zero) Init(1) Warning
 Warn about compile-time integer division by zero
 
 Wduplicated-cond
-C ObjC C++ ObjC++ Var(warn_duplicated_cond) Warning LangEnabledBy(C ObjC C++ ObjC++,Wall)
+C ObjC C++ ObjC++ Var(warn_duplicated_cond) Init(0) Warning
 Warn about duplicated conditions in an if-else-if chain
 
 Weffc++

diff --git a/gcc/genemit.c b/gcc/genemit.c
index 13f911955bcfdbd96e45b4bcd2beac2648851c5a..e92f75750ecc63b9f443e61240e64cff65718237 100644 (file)
--- a/gcc/genemit.c
+++ b/gcc/genemit.c
@@ -182,7 +182,7 @@ gen_exp (rtx x, enum rtx_code subroutine_type, char *used)
               && INTVAL (x) <= MAX_SAVED_CONST_INT)
        printf ("const_int_rtx[MAX_SAVED_CONST_INT + (%d)]",
                (int) INTVAL (x));
-      else if (STORE_FLAG_VALUE > 1 && INTVAL (x) == STORE_FLAG_VALUE)
+      else if (INTVAL (x) == STORE_FLAG_VALUE)
        printf ("const_true_rtx");
       else
        {

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 58a0289a244aaf1f69c26ad6cc8ab15a5cab037b..8211e03f9437ebef75d4e1c59ed02a77a5512eb8 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2015-10-02  Marek Polacek  <polacek@redhat.com>
+
+       * c-c++-common/Wduplicated-cond-2.c: Skip until PR67819 is resolved.
+
 2015-10-02  Sebastian Pop  <s.pop@samsung.com>
 
        PR tree-optimization/67754

diff --git a/gcc/testsuite/c-c++-common/Wduplicated-cond-2.c b/gcc/testsuite/c-c++-common/Wduplicated-cond-2.c
index 90a866305bff8e078e5bf58cd9dfe7c4a424dfe5..c424e76644289bed57b21c79d1560534809611f2 100644 (file)
--- a/gcc/testsuite/c-c++-common/Wduplicated-cond-2.c
+++ b/gcc/testsuite/c-c++-common/Wduplicated-cond-2.c
@@ -1,6 +1,7 @@
 /* PR c/64249 */
 /* { dg-do compile } */
 /* { dg-options "-Wall" } */
+/* { dg-skip-if "PR67819" { *-*-* } } */
 
 #ifndef __cplusplus
 # define bool _Bool