warn-access: Fix up warning_at arguments
authorJakub Jelinek <jakub@redhat.com>
Fri, 21 Jan 2022 10:18:45 +0000 (11:18 +0100)
committerJakub Jelinek <jakub@redhat.com>
Fri, 21 Jan 2022 10:22:22 +0000 (11:22 +0100)
A warning regression fix I'm about to post warns (and breaks bootstrap due
to that) on the following spot.  Seems it is a copy and paste from
earlier code that mentions the %qD variable instead of talking about
unnamed temporary.

2022-01-21  Jakub Jelinek  <jakub@redhat.com>

* gimple-ssa-warn-access.cc (pass_waccess::warn_invalid_pointer):
Avoid passing var to warning_at when the format string doesn't
refer to it.

gcc/gimple-ssa-warn-access.cc

index c36cd5d..8bc33ee 100644 (file)
@@ -3953,15 +3953,14 @@ pass_waccess::warn_invalid_pointer (tree ref, gimple *use_stmt,
                            "may be used")
                       : G_("using dangling pointer %qE to an unnamed "
                            "temporary")),
-                     ref, var))
+                     ref))
       || (!ref
          && warning_at (use_loc, OPT_Wdangling_pointer_,
                         (maybe
                          ? G_("dangling pointer to an unnamed temporary "
                               "may be used")
                          : G_("using a dangling pointer to an unnamed "
-                              "temporary")),
-                        var)))
+                              "temporary")))))
     {
       inform (DECL_SOURCE_LOCATION (var),
              "unnamed temporary defined here");