This is an archive of the discontinued LLVM Phabricator instance.

Explain --reproduce option.
ClosedPublic

Authored by ruiu on Apr 11 2019, 2:26 AM.

Details

Summary

I think --reproduce is no longer a debug-only option but a useful
option that a common user may want to use. So, this patch updates
the description of the option in the manual page.

Diff Detail

Repository
rL LLVM

Event Timeline

ruiu created this revision.Apr 11 2019, 2:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 11 2019, 2:26 AM
MaskRay accepted this revision.Aug 23 2019, 1:33 AM
MaskRay added subscribers: peter.smith, grimar.

The updated description looks good to me. I even use this option to debug GNU linkers a lot (usually to check why/how they do ABC) because they don't provide this nice feature...

+@grimar @peter.smith in case they have wording suggestions.

This revision is now accepted and ready to land.Aug 23 2019, 1:33 AM
ruiu added a comment.Aug 23 2019, 1:38 AM

Yeah, I actually like this option *a lot*. This is one of the best features we've added to lld.

This revision was automatically updated to reflect the committed changes.

It would be good to get at least the tar file part added to the ld.lld --help option, I remember having to use the file command the first time I used --reproduce and I think more people will use --help than look in the man page.

lld/trunk/docs/ld.lld.1
411

I think this will work fine. A suggestion that gives a bit more detail on the contents, not got a strong opinion so by all means keep the original.

Write a tar file containing all the input files needed to reproduce the link, a text file called response.txt containing the command line options and a text file called version.txt containing the output of ld.lld --version. The archive when unpacked can be used to re-run the linker with the same options and input files.

ruiu added a comment.Aug 23 2019, 3:35 AM

It would be good to get at least the tar file part added to the ld.lld --help option, I remember having to use the file command the first time I used --reproduce and I think more people will use --help than look in the man page.

I like that detailed explanation. I've already submitted this change, but can you submit yours to update it?

It would be good to get at least the tar file part added to the ld.lld --help option, I remember having to use the file command the first time I used --reproduce and I think more people will use --help than look in the man page.

I like that detailed explanation. I've already submitted this change, but can you submit yours to update it?

Sure I'll send out a review in a few moments.