[PATCH] D128055: [llvm-ar] Improve MRI script CREATE command handling

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 23 23:16:00 PDT 2022


jhenderson added a comment.

There's a potentially relevant pre-merge failure in mri-create.test on debian (see above), which you should look at before landing this (I suspect it's the missing %errc substitution I highlighted above).



================
Comment at: llvm/test/tools/llvm-ar/mri-create.test:26
+# RUN: not llvm-ar -M < thin-bad.mri 2>&1 | FileCheck --check-prefix=BAD %s
+# BAD: error: bad/bad.a: no such file or directory
+
----------------
Missing %errc substitution usage here?


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

https://reviews.llvm.org/D128055



More information about the llvm-commits mailing list