On Tue, Aug 30, 2011 at 6:38 PM, Martin Jambor <mjam...@suse.cz> wrote:
>
> Hi,
>
> this is a ping of a patch that I have originally posted almost two
> months ago.
>
> Since (same body) aliases have their own cgraph_nodes, the check for
> them in cgraph_redirect_edge_call_stmt_to_callee is now unnecessary
> because e->callee is now the alias, not the function node.
>
> The following patch therefore removes it.  Bootstrapped and tested
> again on x86_64-linux yesterday, OK for trunk?

Ok.

Thanks,
Richard.

> Thanks,
>
> Martin
>
>
> 2011-07-08  Martin Jambor  <mjam...@suse.cz>
>
>        * cgraphunit.c (cgraph_redirect_edge_call_stmt_to_callee): Alias
>        check removed.
>
> Index: src/gcc/cgraphunit.c
> ===================================================================
> --- src.orig/gcc/cgraphunit.c
> +++ src/gcc/cgraphunit.c
> @@ -2373,9 +2373,7 @@ cgraph_redirect_edge_call_stmt_to_callee
>  #endif
>
>   if (e->indirect_unknown_callee
> -      || decl == e->callee->decl
> -      /* Don't update call from same body alias to the real function.  */
> -      || (decl && cgraph_get_node (decl) == cgraph_get_node 
> (e->callee->decl)))
> +      || decl == e->callee->decl)
>     return e->call_stmt;
>
>  #ifdef ENABLE_CHECKING
>

Reply via email to