[PATCH] D65430: Add `--dependency-files` option, which is equivalent to compiler option -MD.

[Petr Hosek via Phabricator via llvm-commits]

phosek added a comment.

In D65430#1616208 <https://reviews.llvm.org/D65430#1616208>, @MaskRay wrote:

> `#pragma comment(lib, "b.a")` is not common. If the build system goes through the trouble to use deplibs,


These are already being used by libc++ and libc++abi so I wouldn't say it's uncommon.

> there may likely be other post processing steps. I wonder if it can just point `-fuse-ld=` to a custom wrapper that processes `ld.lld -t` output and generates `$out.link.d`.

This sounds like a lot of extra effort that would have to be replicated in every build system that would like to use the depfile support (i.e. every build that wants to use linker depfiles would need such wrapper and integrate it into their build).

The implementation of this feature is very minimal and doesn't significantly increase the linker complexity so I don't understand why there's so much pushback?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65430