On 12/11/15 13:15, Jan Hubicka wrote: >> Jan, >> b) augment can_replace_by_local_alias_in_vtable to check whether >> aliases can be created? > > I think this is best: can_replace_by_local_alias_in_vtable exists to prevent the > body walk in cases we are not going to create the alias. This is because in LTO > we may need to stream in the constructor from the object file that is not copletely > free and thus it is better to not touch it unless necessary. I went with augmenting can_replace_by_local_alias, which can_replace_by_local_alias_in_vtable calls. I also noticed that both should be static, which I suspect will encourage the inliner to go inline them and then determine a bunch of code is unreachable. tested on x86-linux and ptx-none. ok? nathan