[gcc r12-6788] warn-access: Fix up warning_at arguments
Jakub Jelinek
jakub@gcc.gnu.org
Fri Jan 21 10:23:05 GMT 2022
https://gcc.gnu.org/g:3e0b19f1d670aa37ac06d63d059c7277ae1188d8
commit r12-6788-g3e0b19f1d670aa37ac06d63d059c7277ae1188d8
Author: Jakub Jelinek <jakub@redhat.com>
Date: Fri Jan 21 11:18:45 2022 +0100
warn-access: Fix up warning_at arguments
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.
Diff:
---
gcc/gimple-ssa-warn-access.cc | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/gcc/gimple-ssa-warn-access.cc b/gcc/gimple-ssa-warn-access.cc
index c36cd5d45d4..8bc33eeb6fa 100644
--- a/gcc/gimple-ssa-warn-access.cc
+++ b/gcc/gimple-ssa-warn-access.cc
@@ -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");
More information about the Gcc-cvs
mailing list