[PATCH] D117443: [llvm-ar] -s: don't convert a thin archive to a regular one

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

Unrelated to this patch:

  % rm -f x.a && fllvm-ar rcT x.a a.o b.o
  % ar q x.a c.o   # q or r
  ar: Cannot convert existing thin library x.a to normal format

Do we want such a diagnostic? llvm-ar current behavior is to convert the thin archive to a regular one.

The opposite has a diagnostic, like GNU ar.

  % 'rm' -f x.a && fllvm-ar rc x.a a.o b.o
  % fllvm-ar rT x.a c.o
  /tmp/RelA/bin/llvm-ar: error: cannot convert a regular archive to a thin one


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117443