Re: Make ipa-icf to not prevent devirtualization

2016-01-15 Thread Jan Hubicka
> On Fri, Jan 15, 2016 at 12:01 PM, Jan Hubicka wrote: > > Hi, > > this patch avoid ipa-icf to take away body of a virtual function which is a > > subject > > of later devirtualization. > > But isn't can_remove_if_no_direct_alls_p () supposed to return false > then? (depending > on cgraph state

Re: Make ipa-icf to not prevent devirtualization

2016-01-15 Thread Richard Biener
On Fri, Jan 15, 2016 at 12:01 PM, Jan Hubicka wrote: > Hi, > this patch avoid ipa-icf to take away body of a virtual function which is a > subject > of later devirtualization. But isn't can_remove_if_no_direct_alls_p () supposed to return false then? (depending on cgraph state) Richard. > Boo

Make ipa-icf to not prevent devirtualization

2016-01-15 Thread Jan Hubicka
Hi, this patch avoid ipa-icf to take away body of a virtual function which is a subject of later devirtualization. Bootstrapped/regtested x86_64-linux, comitted. Honza PR ipa/68148 * ipa-icf.c (sem_function::merge): Virtual functions may become reachable even if they add