[PATCH] D107471: [llvm-ar] Add some test-cases for empty archives

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/test/tools/llvm-ar/display-empty.test:12
+# RUN: llvm-ar Trc %t/thin-archive-empty.a
+# RUN: llvm-ar t %t/thin-archive-empty.a 2>&1 | tee %t1.log | count 0
+# RUN: llvm-ar tv %t/thin-archive-empty.a 2>&1 | tee %t2.log  | count 0
----------------
Delete ` | tee %t1.log` which is only for debugging


================
Comment at: llvm/test/tools/llvm-ar/display-empty.test:13
+# RUN: llvm-ar t %t/thin-archive-empty.a 2>&1 | tee %t1.log | count 0
+# RUN: llvm-ar tv %t/thin-archive-empty.a 2>&1 | tee %t2.log  | count 0
----------------
With `tv`, `t` coverage may be redundant.


================
Comment at: llvm/test/tools/llvm-ar/print.test:93
+# RUN: llvm-ar Trc %t/thin-archive-empty.a
+# RUN: llvm-ar p %t/thin-archive-empty.a 2>&1 | tee %t1.log | count 0
+# RUN: llvm-ar pv %t/thin-archive-empty.a 2>&1 | tee %t2.log  | count 1
----------------
Delete `| tee %t1.log` which is only for debugging


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

https://reviews.llvm.org/D107471