Issue -Waddress also for reference members [PR96507].
Resolves: PR c++/96507 - missing -Waddress for member references gcc/cp/ChangeLog: PR c++/96507 * typeck.c (warn_for_null_address): Handle reference members. gcc/testsuite/ChangeLog: PR c++/96507 * g++.dg/warn/Waddress-8.C: New test.
This commit is contained in:
parent
30ba058f77
commit
2dd56aed3e
2 changed files with 89 additions and 1 deletions
|
@ -4681,6 +4681,9 @@ warn_for_null_address (location_t location, tree op, tsubst_flags_t complain)
|
||||||
{
|
{
|
||||||
STRIP_NOPS (op);
|
STRIP_NOPS (op);
|
||||||
|
|
||||||
|
if (TREE_CODE (op) == COMPONENT_REF)
|
||||||
|
op = TREE_OPERAND (op, 1);
|
||||||
|
|
||||||
if (DECL_P (op))
|
if (DECL_P (op))
|
||||||
warned = warning_at (location, OPT_Waddress,
|
warned = warning_at (location, OPT_Waddress,
|
||||||
"the compiler can assume that the address of "
|
"the compiler can assume that the address of "
|
||||||
|
|
85
gcc/testsuite/g++.dg/warn/Waddress-8.C
Normal file
85
gcc/testsuite/g++.dg/warn/Waddress-8.C
Normal file
|
@ -0,0 +1,85 @@
|
||||||
|
/* PR c++/96507 - missing -Waddress for member references
|
||||||
|
{ dg-do compile }
|
||||||
|
{ dg-options "-Wall" } */
|
||||||
|
|
||||||
|
typedef void F ();
|
||||||
|
|
||||||
|
extern F 𝔢
|
||||||
|
extern int &eir;
|
||||||
|
|
||||||
|
bool warn_ext_rfun ()
|
||||||
|
{
|
||||||
|
return &efr != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_ext_rvar ()
|
||||||
|
{
|
||||||
|
return &eir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
bool warn_parm_rfun (F &rf)
|
||||||
|
{
|
||||||
|
return &rf != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_parm_rvar (int &ir)
|
||||||
|
{
|
||||||
|
return &ir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
// Comparing the address of a reference argument to null also triggers
|
||||||
|
// a -Wnonnull-compare (that seems like a bug, hence PR 103363).
|
||||||
|
// { dg-prune-output "-Wnonnull-compare" }
|
||||||
|
|
||||||
|
|
||||||
|
struct S
|
||||||
|
{
|
||||||
|
F &fr;
|
||||||
|
int &ir;
|
||||||
|
};
|
||||||
|
|
||||||
|
extern S es, esa[];
|
||||||
|
|
||||||
|
bool warn_ext_mem_rfun ()
|
||||||
|
{
|
||||||
|
return &es.fr != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_ext_mem_rvar ()
|
||||||
|
{
|
||||||
|
return &es.ir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
bool warn_ext_arr_mem_rfun (int i)
|
||||||
|
{
|
||||||
|
return &esa[i].fr != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_ext_arr_mem_rvar (int i)
|
||||||
|
{
|
||||||
|
return &esa[i].ir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
bool warn_parm_mem_rfun (S &s)
|
||||||
|
{
|
||||||
|
return &s.fr != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_parm_mem_rvar (S &s)
|
||||||
|
{
|
||||||
|
return &s.ir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
bool warn_parm_arr_mem_rfun (S sa[], int i)
|
||||||
|
{
|
||||||
|
return &sa[i].fr != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
||||||
|
|
||||||
|
bool warn_parm_arr_mem_rvar (S sa[], int i)
|
||||||
|
{
|
||||||
|
return &sa[i].ir != 0; // { dg-warning "-Waddress" }
|
||||||
|
}
|
Loading…
Add table
Reference in a new issue