Re: [PATCH] Fix PR c++/48838

2011-05-05 Thread Jason Merrill
OK. Jason

Re: [PATCH] Fix PR c++/48838

2011-05-05 Thread Dodji Seketeli
Jason Merrill writes: > On 05/05/2011 03:52 PM, Dodji Seketeli wrote: >> + if (is_overloaded_fn (fn)) >> +fn = get_first_fn (fn); >> + else if (BASELINK_P (fn)) > > is_overloaded_fn should be true for a BASELINK. Correct. I should have seen that. I have bootstrapped the patch below then.

Re: [PATCH] Fix PR c++/48838

2011-05-05 Thread Jason Merrill
On 05/05/2011 03:52 PM, Dodji Seketeli wrote: + if (is_overloaded_fn (fn)) +fn = get_first_fn (fn); + else if (BASELINK_P (fn)) is_overloaded_fn should be true for a BASELINK. Jason

Re: [PATCH] Fix PR c++/48838

2011-05-05 Thread Dodji Seketeli
Jason Merrill writes: > I think using get_first_fn could make non_static_member_function_p a > lot shorter. Like this? Tested like the previous patch. gcc/cp PR c++/48838 * cp-tree.h (non_static_member_function_p): Declare new function. * tree.c (non_static_member_func

Re: [PATCH] Fix PR c++/48838

2011-05-03 Thread Jason Merrill
I think using get_first_fn could make non_static_member_function_p a lot shorter. Jason

[PATCH] Fix PR c++/48838

2011-05-03 Thread Dodji Seketeli
Hello, This is another fallout of the fix for PR c++/47172, where we started to make finish_call_expr make member function call expressions that have a dependent "this" pointer be recognized by type_dependent_expression_p as being type dependent. The problem is that a node being a BASELINK is not