From 207fd2123a05b4d163c949e3cad0fd293d4f45b4 Mon Sep 17 00:00:00 2001 From: Jason Merrill Date: Fri, 19 Sep 2014 20:54:55 -0400 Subject: [PATCH] re PR c++/61465 (Bogus parameter set but not used warning in constructor initialization list) PR c++/61465 * call.c (convert_like_real) [ck_identity]: Call mark_rvalue_use after pulling out an element from a CONSTRUCTOR. From-SVN: r215413 --- gcc/cp/ChangeLog | 6 ++++++ gcc/cp/call.c | 2 +- gcc/testsuite/g++.dg/warn/Wunused-parm-6.C | 8 ++++++++ 3 files changed, 15 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/g++.dg/warn/Wunused-parm-6.C diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index e7e0a3c785b..f5092fb14c5 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2014-09-19 Jason Merrill + + PR c++/61465 + * call.c (convert_like_real) [ck_identity]: Call mark_rvalue_use + after pulling out an element from a CONSTRUCTOR. + 2014-09-18 Jakub Jelinek PR c++/63248 diff --git a/gcc/cp/call.c b/gcc/cp/call.c index 161235b64cb..8f1b91a4f16 100644 --- a/gcc/cp/call.c +++ b/gcc/cp/call.c @@ -6206,7 +6206,6 @@ convert_like_real (conversion *convs, tree expr, tree fn, int argnum, return expr; } case ck_identity: - expr = mark_rvalue_use (expr); if (BRACE_ENCLOSED_INITIALIZER_P (expr)) { int nelts = CONSTRUCTOR_NELTS (expr); @@ -6217,6 +6216,7 @@ convert_like_real (conversion *convs, tree expr, tree fn, int argnum, else gcc_unreachable (); } + expr = mark_rvalue_use (expr); if (type_unknown_p (expr)) expr = instantiate_type (totype, expr, complain); diff --git a/gcc/testsuite/g++.dg/warn/Wunused-parm-6.C b/gcc/testsuite/g++.dg/warn/Wunused-parm-6.C new file mode 100644 index 00000000000..95fb7e2dbe4 --- /dev/null +++ b/gcc/testsuite/g++.dg/warn/Wunused-parm-6.C @@ -0,0 +1,8 @@ +// PR c++/61465 +// { dg-do compile { target c++11 } } +// { dg-options "-Wunused-but-set-parameter" } + +struct Foo { + Foo(void* x) : y{static_cast(x)} {} + char* y; +}; -- 2.30.2