[PATCH] D128838: [llvm-ar][test] Add additional MRI script testing

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 00:38:13 PDT 2022


jhenderson accepted this revision.
jhenderson added a comment.

LGTM, with one nit.



================
Comment at: llvm/test/tools/llvm-ar/mri-end.test:10
+## Empty file.
+# RUN: touch %t/empty.mri
+# RUN: llvm-ar -M < empty.mri
----------------
Also above - you cd'ed to `%t` so don't need to reference paths as `%t/empty.mri` or `%t/elf.o` etc.


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

https://reviews.llvm.org/D128838



More information about the llvm-commits mailing list