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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

Thanks!



================
Comment at: lld/ELF/Driver.cpp:1584
+//
+// When you write a Makefile, you usually write it so that the final
+// executable depends on all user-generated object files. Normally, you
----------------
I hope @mcgrathr can spend a bit more time inspecting the wording here as he is the author of the GNU ld and gold patch.


================
Comment at: lld/test/ELF/dependency-file.s:3
+# RUN: mkdir -p %t
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux %s -o %t/foo.o
+# RUN: llvm-mc -filetype=obj -triple=x86_64-unknown-linux /dev/null -o "%t/bar baz.o"
----------------
`x86_64-unknown-linux` -> `x86_64`


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

https://reviews.llvm.org/D82437