[PATCH] D64330: [llvm-ar][test] Add to MRI test coverage

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 07:02:46 PDT 2019


MaskRay added inline comments.


================
Comment at: test/tools/llvm-ar/quick-append.test:94
+    Section: .text
\ No newline at end of file

----------------
`No newline at end of file`


================
Comment at: test/tools/llvm-ar/read-only-archive.py:11
+## Make read only:
+# RUN: %python %s %t/archive.a
+
----------------
`chmod 444 %t/archive.a` and rename this file from `.py` to `.test`. I'm not sure if a `.py` test will run in this directory


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64330





More information about the llvm-commits mailing list