[PATCH] D80838: [llvm-ar] Add more tests for errors in opening archives
Owen Reynolds via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 1 08:00:41 PDT 2020
gbreynoo added a comment.
In D80838#2065845 <https://reviews.llvm.org/D80838#2065845>, @jhenderson wrote:
> The test you have modified is for testing the "p" (print) operation, but your new test cases are not really about that option, but rather about generic file handling. I'm not convinced they belong in this test file, but @gbreynoo might have other opinions, since he's the one who's been working on the llvm-ar testing recently.
>
> Could you clarify why you are adding these test cases? I don't think the behaviour is really part of llvm-ar, but rather part of the underlying support libraries which control when a file can be opened or not.
I agree with James that these may be best placed in their own test. Currently there are individual tests for invalid-object-file.test, missing-thin-archive-member.test etc.
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