[PATCH] D36311: [ThinLTO] Add GraphTraits for FunctionSummaries
Charles Saternos via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 25 08:01:19 PDT 2017
ncharlie marked 11 inline comments as done.
ncharlie added inline comments.
================
Comment at: test/ThinLTO/X86/module_summary_graph_traits.ll:32
+define void @l2() {
+ call void @l1()
+ ret void
----------------
Despite the indirect recursion in l1/l2, the SCCs only have 1 node in each. The loop is still detected (with the scc_iterator::hasLoop function), but there's an SCC for each function.
https://reviews.llvm.org/D36311
More information about the llvm-commits
mailing list