[PATCH] D82437: [ELF] Add --dependency-file option

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 24 10:15:32 PDT 2020


MaskRay added a comment.

In D82437#2110769 <https://reviews.llvm.org/D82437#2110769>, @phosek wrote:

> This is an updated version of D65430 <https://reviews.llvm.org/D65430> which matches the feature implemented by ld and gold in https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=f37b21b481a7804a13c5806098c19b6119288ba4. I'm happy to abandon this change in favor of D65430 <https://reviews.llvm.org/D65430> if @ruiu wants to go ahead with his change.


You can just paste https://sourceware.org/bugzilla/show_bug.cgi?id=22843 beside "ld and gold which gained this feature recently" in the description.

For the tests, you can read `reproduce*` and make a summary.



================
Comment at: lld/ELF/Options.td:135
 
+defm dependency_file: Eq<"dependency-file", "Write a dependency file">,
+  MetaVarName<"<path>">;
----------------
`EEq`

We don't want to support `-dependency-file`


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

https://reviews.llvm.org/D82437





More information about the llvm-commits mailing list