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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 3 16:01:35 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.

LGTM.



================
Comment at: lld/ELF/Options.td:136
+defm dependency_file: EEq<"dependency-file", "Write a dependency file">,
+  MetaVarName<"<path>">;
+
----------------
I wonder whether `<file>` will work better than `<path>` because `file` conveys more information: a path can be either a file or a directory (if we exclude other possibilities).


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

https://reviews.llvm.org/D82437



More information about the llvm-commits mailing list