[PATCH] D34063: [ThinLTO][llvm-lto2] Dump YAML for module summaries

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 9 13:05:53 PDT 2017


pcc added a comment.

Can you split this into two patches (both with their own tests), one which adds the subcommand and the other which adds the new YAML output?



================
Comment at: tools/llvm-dis/llvm-dis.cpp:155
 
+void printModuleSummaryIndex(raw_ostream &Out, const ModuleSummaryIndex *S) {
+  Out << "; Module summary:\n";
----------------
This seems like an unrelated change.


https://reviews.llvm.org/D34063





More information about the llvm-commits mailing list