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

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

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 ebotca...@adacore.com * 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.

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 ebotca...@adacore.com * 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