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

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 8 15:40:37 PDT 2017


ncharlie added inline comments.


================
Comment at: include/llvm/IR/ModuleSummaryIndex.h:871
+  static NodeRef getEntryNode(ModuleSummaryIndex *I) {
+    return *I->functions_begin();
+  }
----------------
This entry node isn't necessarily at the bottom of the callgraph. I'm not sure how to address this problem at the moment. 


https://reviews.llvm.org/D36311





More information about the llvm-commits mailing list