[PATCH] D79513: [dsymutil] Print statistics about debug_info reduction in verbose mode.

[Frederic Riss via Phabricator via llvm-commits]

friss added inline comments.


================
Comment at: llvm/lib/DWARFLinker/DWARFLinker.cpp:2533
 
+  if (Options.Verbose) {
+    // Create a vector sorted in descending order by output size.
----------------
JDevlieghere wrote:
> friss wrote:
> > Should we be using a different `--stat` flag for this? The verbose flag will output so much info on a real link... 
> Not to mention that verbose mode implies single threaded, which is the reason I put it under verbose. If we want to use `--stats` we'll need a way to synchronize the printing across architectures for fat binaries. Do you think that's worth it? 
Do we need to synchronize? Maybe we should just print the current architecture at the beginning of the line to disambiguate?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79513/new/

https://reviews.llvm.org/D79513