> > The cgraph.cc change was what I actually needed for the fix, the > cgraphclones.cc was only because I've noticed that it constructs a new > node (so is initialized to whatever random flag_semantic_interposition is > right now) and initializing it to what it is cloned from made more sense.
OK, thanks. It only is needed for nodes which definition flag and public linkage, so should not need to copy in cgraph clones and there are other places that creates new nodes (late function etc). I will move the logic to visibility pass and to add_new_function and also kill the constructor. I originally intended to set it at the consturction time but forget to think of the frotnends changing opt_for_fn later from the optimization attribute. This also makes me wonder if C++ FE updates the implicit aliases once they have been created... Honza > > Jakub >
