[PATCH] D63935: [llvm-ar][test] Increase llvm-ar test coverage

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 5 01:02:59 PDT 2019


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


================
Comment at: test/tools/llvm-ar/extract.test:4
+
+# Extracting from an empty archive should not warn or error:
+RUN: llvm-ar cr %t/empty.a
----------------
`## `


================
Comment at: test/tools/llvm-ar/insert-after.test:18
+
+# Insert file at back:
+RUN: rm -f %t-back.ar
----------------
`## `


================
Comment at: test/tools/llvm-ar/insert-before.test:8
+
+# Insert one file:
+RUN: rm -f %t-one.ar
----------------
`## `


================
Comment at: test/tools/llvm-ar/move-before.test:8
+
+# Move one file:
+RUN: rm -f %t-one.ar
----------------
`## `


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

https://reviews.llvm.org/D63935





More information about the llvm-commits mailing list