> Hi,
> 
> as discussed with Honza on many occasions, all users of
> cgraph_get_create_node really want cgraph_get_create_real_symbol_node,
> i.e. they are not interested in inline nodes and should get a
> standalone node instead.  So this patch changes cgraph_get_create_node
> to do what cgraph_get_create_real_symbol_node currently does and
> removes the latter altogether.
> 
> I had to change a call to cgraph_get_create_node to cgraph_get_node in
> lto-streamer-in.c so that it does not make the node it operates on a
> clone of another one because this made ipa_pta_execute abort on assert
> after calling cgraph_get_body (visionary points to Richi for putting
> the assert there).
> 
> The patch successfully passed bootstrap and testing ("all" languages +
> Ada) and LTO-bootstrap (C and C++ only) on x86_64-linux.
> 
> OK for trunk?
> 
> Thanks,
> 
> Martin
> 
> 
> 2013-11-12  Martin Jambor  <mjam...@suse.cz>
> 
>       * cgraph.c (cgraph_get_create_node): Do what
>       cgraph_get_create_real_symbol_node used to do.
>       (cgraph_get_create_real_symbol_node): Removed.  Changed all users to
>       call cgraph_get_create_node.
>       * cgraph.h (cgraph_get_create_real_symbol_node): Removed.
>       * lto-streamer-in.c (input_function): Call cgraph_get_node instead of
>       cgraph_get_create_node.  Assert we get a node.

OK,
thanks!
Honza

Reply via email to