Add -Wduplicated-cond

This adds -Wduplicated-cond to warnings.m4.  This caught one bug.

I tried adding -Wduplicated-branches as well, but it results in some
spurious failures from code like this in cgen.h:

    #define CGEN_ATTR_TYPE(n) \
    struct { unsigned int bool_; \
	     CGEN_ATTR_VALUE_TYPE nonbool[(n) ? (n) : 1]; }

This will trigger a warning if passed n==1, which seems like a
perfectly valid thing to do; and there were other issues like this as
well.

ChangeLog
2018-05-07  Tom Tromey  <tom@tromey.com>

	* configure: Rebuild.
	* warning.m4 (AM_GDB_WARNINGS): Add -Wduplicated-cond.

gdbserver/ChangeLog
2018-05-07  Tom Tromey  <tom@tromey.com>

	* configure: Rebuild.
This commit is contained in:
Tom Tromey
2018-04-21 12:00:29 -06:00
parent ce887586b4
commit aff689d36d
5 changed files with 15 additions and 3 deletions

View File

@ -1,3 +1,8 @@
2018-05-07 Tom Tromey <tom@tromey.com>
* configure: Rebuild.
* warning.m4 (AM_GDB_WARNINGS): Add -Wduplicated-cond.
2018-05-07 Tom Tromey <tom@tromey.com>
PR tdep/20362:

3
gdb/configure vendored
View File

@ -15367,7 +15367,8 @@ build_warnings="-Wall -Wpointer-arith \
-Wno-mismatched-tags \
-Wno-error=deprecated-register \
-Wsuggest-override \
-Wimplicit-fallthrough"
-Wimplicit-fallthrough \
-Wduplicated-cond"
case "${host}" in
*-*-mingw32*)

View File

@ -1,3 +1,7 @@
2018-05-07 Tom Tromey <tom@tromey.com>
* configure: Rebuild.
2018-05-04 Tom Tromey <tom@tromey.com>
* configure: Rebuild.

View File

@ -7174,7 +7174,8 @@ build_warnings="-Wall -Wpointer-arith \
-Wno-mismatched-tags \
-Wno-error=deprecated-register \
-Wsuggest-override \
-Wimplicit-fallthrough"
-Wimplicit-fallthrough \
-Wduplicated-cond"
case "${host}" in
*-*-mingw32*)

View File

@ -44,7 +44,8 @@ build_warnings="-Wall -Wpointer-arith \
-Wno-mismatched-tags \
-Wno-error=deprecated-register \
-Wsuggest-override \
-Wimplicit-fallthrough"
-Wimplicit-fallthrough \
-Wduplicated-cond"
case "${host}" in
*-*-mingw32*)