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

[Owen Reynolds via Phabricator via llvm-commits]

gbreynoo updated this revision to Diff 422524.
gbreynoo added a comment.

Corrected comments including replacing use of "full archive" with "regular archive" as suggested. To remain consistent two tests have been renamed to use the term "regular". The checks for "thinness" in `llvm-ar.cpp` now only applies for write commands.


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

https://reviews.llvm.org/D123142

Files:
  llvm/test/tools/llvm-ar/flatten-thin-archive.test
  llvm/test/tools/llvm-ar/full-to-thin-archive.test
  llvm/test/tools/llvm-ar/regular-to-thin-archive.test
  llvm/test/tools/llvm-ar/thin-to-full-archive.test
  llvm/test/tools/llvm-ar/thin-to-regular-archive.test
  llvm/tools/llvm-ar/llvm-ar.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123142.422524.patch
Type: text/x-patch
Size: 7910 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220413/80545a3f/attachment.bin>