[PATCH] D58599: [LLD] Add summary support

Wed Feb 27 13:56:14 PST 2019

rnk added a comment.

In D58599#1410086 <https://reviews.llvm.org/D58599#1410086>, @aganea wrote:> The following __does not__ produce the expected result - instead the text is half-printed on the console, and the dump file only contains partial information (Discarded symbols and the cmd-line):

>   lld-link.exe ... /verbose >dump.txt
>

Because, of course, we log to stderr. =p I think it's just a holdover convention from the compiler, where everything must be sent to stderr in case the user puts `-o -` on the command line, which would end up mixing diagnostics with binary output. This gets set here:
https://github.com/llvm/llvm-project/blob/master/lld/include/lld/Common/ErrorHandler.h#L89

You could probably change the convention for lld-link, but maybe it's best to leave ELF alone.

Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D58599