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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 29 22:48:51 PDT 2019


MaskRay added inline comments.


================
Comment at: lld/ELF/Config.h:87
   llvm::StringMap<uint64_t> sectionStartMap;
+  llvm::StringSet<> dependencyFiles; // for --write-dependencies
   llvm::StringRef chroot;
----------------
Move `dependencyFiles` below `chroot`?


================
Comment at: lld/ELF/Driver.cpp:1393
+// because you don't know the exact paths of libraries, even though
+// statically-linked system libraries are technically a part of your
+// program. By using --write-dependencies option, you can make lld to dump
----------------
`statically-linked` -> ``

a.o/b.a/c.so are not statically linked..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65430





More information about the llvm-commits mailing list