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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 10:36:21 PDT 2022


MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/test/tools/llvm-ar/mri-delete.test:10
+# RUN: llvm-ar -M < delete.mri
+# RUN: llvm-ar tv delete.ar | FileCheck %s --check-prefixes=DELETE  --implicit-check-not=elf.o
+
----------------
excess space before --implicit-check-not=elf.o

ditto below


================
Comment at: llvm/test/tools/llvm-ar/mri-end.test:44
+
+#--- empty.mri
+
----------------
empty.mri has two lines. If you want to test an empty script, you can `touch` it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128838



More information about the llvm-commits mailing list