diff options
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/cp/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/cp/cvt.c | 10 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/warn/noeffect8.C | 3 |
3 files changed, 14 insertions, 4 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 6e20eb1..f565f4c 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,8 @@ +2006-09-09 Jason Merrill <jason@redhat.com> + + PR c++/28996 + * cvt.c (convert_to_void): Strip COMPONENT_REF to functions. + 2006-09-08 Volker Reichelt <reichelt@igpm.rwth-aachen.de> PR c++/28858 diff --git a/gcc/cp/cvt.c b/gcc/cp/cvt.c index fea5768..6f61218 100644 --- a/gcc/cp/cvt.c +++ b/gcc/cp/cvt.c @@ -927,9 +927,13 @@ convert_to_void (tree expr, const char *implicit) expr = void_zero_node; } else if (implicit && probe == expr && is_overloaded_fn (probe)) - /* Only warn when there is no &. */ - warning (0, "%s is a reference, not call, to function %qE", - implicit, expr); + { + /* Only warn when there is no &. */ + warning (0, "%s is a reference, not call, to function %qE", + implicit, expr); + if (TREE_CODE (expr) == COMPONENT_REF) + expr = TREE_OPERAND (expr, 0); + } } if (expr != error_mark_node && !VOID_TYPE_P (TREE_TYPE (expr))) diff --git a/gcc/testsuite/g++.dg/warn/noeffect8.C b/gcc/testsuite/g++.dg/warn/noeffect8.C index 99d3688..4eb7f1b 100644 --- a/gcc/testsuite/g++.dg/warn/noeffect8.C +++ b/gcc/testsuite/g++.dg/warn/noeffect8.C @@ -1,4 +1,4 @@ -// PR c++/26696 +// PR c++/26696, 28996 struct A { @@ -9,4 +9,5 @@ int main() { A a; a.f; // { dg-warning "not call" } + A().f; // { dg-warning "not call" } } |