[PATCH] D143600: [ELF][docs] Mention LLD_REPRODUCE and LLD_VERSION
Peter Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 9 02:00:50 PST 2023
peter.smith accepted this revision.
peter.smith added a comment.
This revision is now accepted and ready to land.
LGTM, a couple of small suggestions.
================
Comment at: lld/docs/ld.lld.1:7
.\" from FreeBSD.
-.Dd May 12, 2019
+.Dd Feb 9, 2022
.Dt LD.LLD 1
----------------
Did you mean Feb 9, 2023?
================
Comment at: lld/docs/ld.lld.1:892
+.Cm .comment
+indicating its version. The version string can be overridden by this environment variable.
+
----------------
I recommend changing "indicating its version" to "containing the LLD version string."
The comment for getVersion() says:
```
// Check LLD_VERSION first for ease of testing. // You can get consistent output by using the environment variable. // This is only for testing.
```
perhaps worth adding. "This option is useful to eliminate differences in the binary caused by LLD version number differences."
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143600/new/
https://reviews.llvm.org/D143600
More information about the llvm-commits
mailing list