[PATCH] D80838: [llvm-ar] Add more tests for errors in opening archives

Galina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 27 00:49:54 PDT 2020


gkistanova requested changes to this revision.
gkistanova added a comment.
This revision now requires changes to proceed.

Hi Sameer,

Could you either teach the `error-opening-directory.test` that adirectory could be opened for reading just fine, thus the error message would be different than what you expect here, or handle that in llvm-ar to make sure the error message is reliable, please?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80838





More information about the llvm-commits mailing list