re PR c++/79184 (-Wint-in-bool-context triggered erroneously in template parameter)

	PR c++/79184
	* cvt.c (ocp_convert): Add a sentinel against -Wint-in-bool-context
	if warnings shouldn't be given.

	* g++.dg/warn/Wint-in-bool-context-1.C: New.

From-SVN: r245335

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 7f113ecf4903dbba49759500f11c2adb2276c2ba..5ea0c930a096305bfa378c84a10125aac412722f 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -4,6 +4,10 @@
        * pt.c (type_dependent_expression_p): Check if the expression type
        is null.
 
+       PR c++/79184
+       * cvt.c (ocp_convert): Add a sentinel against -Wint-in-bool-context
+       if warnings shouldn't be given.
+
 2017-02-10  Paolo Carlini  <paolo.carlini@oracle.com>
 
        PR c++/71737

diff --git a/gcc/cp/cvt.c b/gcc/cp/cvt.c
index ae9991a106c6c60b84905c25d8cc24152f56f1cb..5f4b5e30a5fe2f0e893fbb9501d9165f3ceb6a65 100644 (file)
--- a/gcc/cp/cvt.c
+++ b/gcc/cp/cvt.c
@@ -798,7 +798,15 @@ ocp_convert (tree type, tree expr, int convtype, int flags,
             to the underlying type first.  */
          if (SCOPED_ENUM_P (intype) && (convtype & CONV_STATIC))
            e = build_nop (ENUM_UNDERLYING_TYPE (intype), e);
-         return cp_truthvalue_conversion (e);
+         if (complain & tf_warning)
+           return cp_truthvalue_conversion (e);
+         else
+           {
+             /* Prevent bogus -Wint-in-bool-context warnings coming
+                from c_common_truthvalue_conversion down the line.  */
+             warning_sentinel w (warn_int_in_bool_context);
+             return cp_truthvalue_conversion (e);
+           }
        }
 
       converted = convert_to_integer_maybe_fold (type, e, dofold);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0c17a0362deb84e1016c691b39510fb01d60ad9e..aab9ef51d0acedbe3e1896bc530b37c8cd37795a 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -3,6 +3,9 @@
        PR c++/79435
        * g++.dg/cpp1y/pr79435.C: New.
 
+       PR c++/79184
+       * g++.dg/warn/Wint-in-bool-context-1.C: New.
+
 2017-02-10  Christophe Lyon  <christophe.lyon@linaro.org>
 
        * gcc.target/aarch64/advsimd-intrinsics/p64_p128.c

diff --git a/gcc/testsuite/g++.dg/warn/Wint-in-bool-context-1.C b/gcc/testsuite/g++.dg/warn/Wint-in-bool-context-1.C
new file mode 100644 (file)
index 0000000..117eca4
--- /dev/null
+++ b/gcc/testsuite/g++.dg/warn/Wint-in-bool-context-1.C
@@ -0,0 +1,16 @@
+// PR c++/79184
+// { dg-do compile }
+// { dg-options "-Wint-in-bool-context" }
+
+enum { E = 2 };
+template <bool> void f(int) { }
+template <int> void f() {}
+
+int
+main ()
+{
+  f<1 * 1>(); // { dg-bogus "in boolean context" }
+  f<1 << 1>(); // { dg-bogus "in boolean context" }
+  f<1 ? 3 : 2>(); // { dg-bogus "in boolean context" }
+  f<E>(); // { dg-bogus "in boolean context" }
+}