[PATCH] D61674: [llvm-objcopy] Improve error message for unrecognised archive member

[James Henderson via Phabricator via llvm-commits]

jhenderson marked 5 inline comments as done.
jhenderson added inline comments.


================
Comment at: test/tools/llvm-objcopy/ELF/archive-unknown-members.test:2
+## Show that llvm-objcopy and llvm-strip emit errors when attempting to modify
+## archives containing unrecognised files, and do not modify anything. We use
+## --strip-debug so that the section should be removed, and a difference
----------------
grimar wrote:
> unrecognised -> unrecognized I think?
Thanks. Stupid American spelling ;)


================
Comment at: test/tools/llvm-objcopy/ELF/archive-unknown-members.test:26
+
+# RUN: not llvm-objcopy --strip-debug %t.thin1.a %t2.a 2>&1 \
+# RUN:   | FileCheck %s --check-prefix=THIN -DARCHIVE=%t.thin1.a -DMEMBER=%s
----------------
grimar wrote:
> You have `%t.thin1.a %t2.a` for llvm-objcopy call and just ` %t.thin2.a` for llvm-strip.
> I think you placed `%t2.a` by mistake?
> 
Oops, thanks.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61674