[PATCH] D36311: [ThinLTO] Add GraphTraits for FunctionSummaries

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 13:16:44 PDT 2017


ncharlie added a comment.

I don't think I can use a topological sort in this case since the FunctionSummary callgraph isn't necessarily a DAG, so if there's a cycle it would make the sort useless.

Instead, I tried using the main function as the entry node and that worked. Any suggestions on a clean way of implementing this rather than hard coding a lookup for "main" in the Index?


https://reviews.llvm.org/D36311





More information about the llvm-commits mailing list