Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-17 Thread Eric Botcazou
> Yes, this looks OK. Inliner coul also take care to turn the master clone > into unanalyzed node as remove_unreachable_nodes would, but I do not think > it is worth the effort. Please put the loop later in the function so it > does not slow things down unnecesarily (there are often many inline clo

Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-16 Thread Eric Botcazou
> Yes, this looks OK. Inliner coul also take care to turn the master clone > into unanalyzed node as remove_unreachable_nodes would, but I do not think > it is worth the effort. Please put the loop later in the function so it > does not slow things down unnecesarily (there are often many inline clo

Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-15 Thread Jan Hubicka
> Jan, any opinion on the following patch? > > > 2014-10-06 Eric Botcazou > > > > * ipa-inline-transform.c (can_remove_node_now_p_1): Return false for a > > master clone that has non-inline clones. > > https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00436.html > > Thanks in advance. Y

Re: [patch] Fix ICE during LTRANS on big Ada application

2014-10-15 Thread Eric Botcazou
Jan, any opinion on the following patch? > 2014-10-06 Eric Botcazou > > * ipa-inline-transform.c (can_remove_node_now_p_1): Return false for a > master clone that has non-inline clones. https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00436.html Thanks in advance. -- Eric Botcazou