[PATCH] D34080: [ThinLTO] Add dump-summary command to llvm-lto2 tool

[Davide Italiano via Phabricator via llvm-commits]

davide added a comment.

Minor comment, LGTM with that (once your other patch goes in).



================
Comment at: tools/llvm-lto2/llvm-lto2.cpp:356-367
+static int dumpSummary(int argc, char **argv) {
+  for (StringRef F : make_range(argv + 1, argv + argc)) {
+    std::unique_ptr<MemoryBuffer> MB = check(MemoryBuffer::getFile(F), F);
+    std::unique_ptr<ModuleSummaryIndex> Index =
+        check(getModuleSummaryIndex(*MB), F);
+
+    yaml::Output Out(outs());
----------------
Add a comment to this function.


https://reviews.llvm.org/D34080