[Patch c++] (__attribute__((deprecated)), part a) Fix PR17729

[Jason Merrill]

On 06/08/2010 07:13 PM, IainS wrote:
> the question then, is how to prevent the duplicates, since those places
> are called from more than one part of the process (or, at least, they
> were when I last looked).

For non-function members, I think removing the call in 
finish_class_member_access_expr ought to do the trick, letting 
build_class_member_access_expr handle the warning.

For functions, perhaps just warning in mark_used would work.  Then 
removing the calls in build_over_call and build_call_a and conditioning 
the one in finish_id_expression on !is_overloaded_fn.

Jason