[PATCH] D56046: Output ELF files after ThinLTO is run.

Bill Wendling via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 27 14:38:58 PST 2018


void added a comment.

In D56046#1341491 <https://reviews.llvm.org/D56046#1341491>, @ruiu wrote:

> Replicating the gold behavior sounds fine, but does this patch work? This change seems to leave Buf uninitialized while Buf will be used later. Also, in gold LTO plugin, -obj-path overwrites -save-temps, but this change doesn't seem to replicate that behavior. Could you add tests?


I'll add a test in a bit. I tested it and it did work, but I may have missed something. Let me get the test in and then you can take a look afterwards. Thanks!


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D56046





More information about the llvm-commits mailing list