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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 30 18:55:23 PDT 2019


ruiu marked 2 inline comments as done.
ruiu added inline comments.


================
Comment at: lld/test/ELF/write-dependencies.s:6
+// RUN: ld.lld -o %t/foo.exe %t/foo.o %t/"bar baz.o" --write-dependencies=%t/foo.d
+// RUN: FileCheck %s < %t/foo.d
+
----------------
grimar wrote:
> Since you're testing the output in a particular format, what about adding `--strict-whitespace --match-full-lines` to `FileCheck` invocation?
Since whitespace characters are not significant, I don't think we need to test that, but adding `--match-full-lines` is a good idea.


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