[PATCH] D123142: [llvm-ar] Fix thin archive being wrongly converted to a full archive

[Owen Reynolds via Phabricator via llvm-commits]

gbreynoo added a comment.

The following tests have been modified:

- `flatten-thin-archive.test` now explicitly checks that `%t.a` is thin after the other archives have been added.
- `full-to-thin-archive.test` uses `Inputs/a.txt` as an input rather than itself as spurious passes could have occurred when looking for strings contained in the archive file. Corrected a comment, added a test case for an existing archive and a case attempting to convert an existing archive.
- `thin-to-full-archive.test` now tests the contents to ensure the thin archives member have been added and not the archive itself. Added a test case for an existing archive and a case attempting to convert an existing archive.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123142