[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