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

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 15 10:15:01 PDT 2017


ncharlie added inline comments.


================
Comment at: include/llvm/IR/ModuleSummaryIndex.h:875
+
+  static nodes_iterator nodes_begin(ModuleSummaryIndex *I) {
+    return I->functions_begin();
----------------
Looking more closely at the scc_iterator code, I don't think I need to describe nodes_begin/nodes_end. If that's the case I'll remove them (I'd like to get rid of my fsummary_iterator implementation if possible)


https://reviews.llvm.org/D36311





More information about the llvm-commits mailing list