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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: lld/test/ELF/dependency-file.s:7
+# RUN: FileCheck --match-full-lines %s < %t/foo.d
+
+# CHECK:      {{.*}}foo.exe: \
----------------
Backslashes and # are not tested. If they can be problems on Windows, just add `UNSUPPORTED: system-windows`


================
Comment at: lld/test/ELF/dependency-file.s:8
+
+# CHECK:      {{.*}}foo.exe: \
+# CHECK-NEXT:   {{.*}}foo.o \
----------------
Can you use `FileCheck -DFILE=%t` so that the CHECK lines can test the full paths?


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

https://reviews.llvm.org/D82437