Home

[LCG] Add the first round of mutation support to the lazy call graph.

Press ? to show keyboard shortcuts.
Committed
chandlercApr 23 2014, 4:03 AM
Parents
rL206967: [ARM64] Fix formatting.
Branches
Unknown
Tags
Unknown
Subscribers
None
Projects
None
Description

[LCG] Add the first round of mutation support to the lazy call graph.
This implements the core functionality necessary to remove an edge from
the call graph and correctly update both the basic graph and the SCC
structure. As part of that it has to run a tiny (in number of nodes)
Tarjan-style DFS walk of an SCC being mutated to compute newly formed
SCCs, etc.

This is *very rough* and a WIP. I have a bunch of FIXMEs for code
cleanup that will reduce the boilerplate in this change substantially.
I also have a bunch of simplifications to various parts of both
algorithms that I want to make, but first I'd like to have a more
holistic picture. Ideally, I'd also like more testing. I'll probably add
quite a few more unit tests as I go here to cover the various different
aspects and corner cases of removing edges from the graph.

Still, this is, so far, successfully updating the SCC graph in-place
without disrupting the identity established for the existing SCCs even
when we do challenging things like delete the critical edge that made an
SCC cycle at all and have to reform things as a tree of smaller SCCs.
Getting this to work is really critical for the new pass manager as it
is going to associate significant state with the SCC instance and needs
it to be stable. That is also the motivation behind the return of the
newly formed SCCs. Eventually, I'll wire this all the way up to the
public API so that the pass manager can use it to correctly re-enqueue
newly formed SCCs into a fresh postorder traversal.

rL206968

llvm/trunk/include/llvm/Analysis/LazyCallGraph.h

Loading...

llvm/trunk/lib/Analysis/LazyCallGraph.cpp

Loading...

llvm/trunk/unittests/Analysis/LazyCallGraphTest.cpp

Loading...

Add Comment