Fix PR c++/96310 - Ignoring -Wnonnull via pragma gcc diagnostics still produces an...
authorMartin Sebor <msebor@redhat.com>
Sat, 25 Jul 2020 20:21:47 +0000 (14:21 -0600)
committerMartin Sebor <msebor@redhat.com>
Sat, 25 Jul 2020 20:24:24 +0000 (14:24 -0600)
gcc/c-family/ChangeLog:
PR c++/96310
* c-common.c (check_nonnull_arg): Print note only when warning was
issued.

gcc/c-family/c-common.c
gcc/testsuite/g++.dg/warn/Wnonnull8.C [new file with mode: 0644]

index 51ecde69f2d44479b573e0578b3d81059abe4da8..e2569c65827587d4d8a847bab71e35f8701f42dc 100644 (file)
@@ -5538,7 +5538,7 @@ check_nonnull_arg (void *ctx, tree param, unsigned HOST_WIDE_INT param_num)
     {
       warned = warning_at (loc, OPT_Wnonnull,
                           "%qs pointer null", "this");
-      if (pctx->fndecl)
+      if (warned && pctx->fndecl)
        inform (DECL_SOURCE_LOCATION (pctx->fndecl),
                "in a call to non-static member function %qD",
                pctx->fndecl);
@@ -5548,7 +5548,7 @@ check_nonnull_arg (void *ctx, tree param, unsigned HOST_WIDE_INT param_num)
       warned = warning_at (loc, OPT_Wnonnull,
                           "argument %u null where non-null expected",
                           (unsigned) param_num);
-      if (pctx->fndecl)
+      if (warned && pctx->fndecl)
        inform (DECL_SOURCE_LOCATION (pctx->fndecl),
                "in a call to function %qD declared %qs",
                pctx->fndecl, "nonnull");
diff --git a/gcc/testsuite/g++.dg/warn/Wnonnull8.C b/gcc/testsuite/g++.dg/warn/Wnonnull8.C
new file mode 100644 (file)
index 0000000..7820b2e
--- /dev/null
@@ -0,0 +1,20 @@
+/* PR c++/96310 - Ignoring -Wnonnull via pragma gcc diagnostics still produces
+   an unwanted note
+   { dg-do compile }
+   { dg-options "-Wall" } */
+
+struct C {
+  void f ();                  // { dg-message "in a call" }
+  void g ();                  // { dg-bogus "in a call" }
+};
+
+void f ()
+{
+  static_cast<C*>(0)->f ();   // { dg-warning "\\\[-Wnonnull" }
+}
+
+void g ()
+{
+#pragma GCC diagnostic ignored "-Wnonnull"
+  static_cast<C*>(0)->g ();
+}