[PATCH] D128067: [llvm-ar] Fix MRI ADDLIB command when used with thin archives

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 23 23:19:39 PDT 2022


jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-ar/mri-thin-archive.test:16
+# THIN: !<thin>
+# THIN: elf.o
+# THIN-NEXT: addlib/elf.o/
----------------
Perhaps worth adding `--match-full-lines` to the FileCheck output, to ensure your path is as expected here? (e.g. this will also match `addlib/elf.o/` or `myelf.other.o` etc as things stand)


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

https://reviews.llvm.org/D128067



More information about the llvm-commits mailing list