[llvm-commits] [llvm] r56887 - /llvm/trunk/lib/Transforms/IPO/Internalize.cpp

Duncan Sands baldrick at free.fr
Wed Oct 1 02:19:05 PDT 2008


> >> add preserversCFG() + preservers(CallGraph)
> > 
> > it does not preserve the callgraph, because
> > for a function F that is made internal, before
> > there was an edge to it from the external node,
> > while after there should not be (but there still
> > is - i.e. the callgraph was not updated properly).
> 
> Ah, right, makes sense :)

That said, it would be easy to update the callgraph.

> Thanks for reviewing the patch,

Thanks for fixing up the "preserves" stuff!

Duncan.



More information about the llvm-commits mailing list