[llvm-objcopy][test] Add a test to show that argv[0] is included in error/warning messages

[MaskRay]

Summary

test/llvm-objcopy/ELF/error-format.test is similar to test/llvm-readobj/error-format.test added in D66425.

[grimar]

LGTM.

test/tools/llvm-objcopy/ELF/error-format.test

18 ↗

(On Diff #216146)

Seems not important at all, but EM_RISCV? I'd use something more common instead, probably.

[jhenderson]

test/tools/llvm-objcopy/ELF/error-format.test

9 ↗

(On Diff #216146)

Maybe worth a comment saying why you don't test llvm-objcopy too?

[MaskRay]

+ ## Currently llvm-objcopy does not issue warnings, so it is not tested.

[MaskRay]

test/tools/llvm-objcopy/ELF/error-format.test

18 ↗

(On Diff #216146)

I think this is probably not important... It just adds some variety for unimportant fields..

[grimar]

test/tools/llvm-objcopy/ELF/error-format.test

18 ↗

(On Diff #216146)

OK, I have no strong opinion about this (just EM_RISCV made me wonder why it was used, but it is fine generally).

[jhenderson]

LGTM.

llvm/trunk/test/tools/llvm-objcopy/ELF/error-format.test

+## This test shows that we include the tool name in error/warning messages.
+
+# RUN: not llvm-objcopy %S/non-existent 2>&1 | FileCheck --check-prefix=ERR %s -DTOOL=objcopy
+# RUN: not llvm-strip %S/non-existent 2>&1 | FileCheck --check-prefix=ERR %s -DTOOL=strip
+
+# ERR: llvm-[[TOOL]]{{(\.exe)?}}: error: '{{.*}}': {{[Nn]}}o such file or directory
+
+## Currently llvm-objcopy does not issue warnings, so it is not tested.
+
+# RUN: yaml2obj %s -o %t
+# RUN: llvm-strip %t %t 2>&1 | FileCheck --check-prefix=WARN %s
+
+# WARN: llvm-strip{{(\.exe)?}}: warning: '{{.*}}' was already specified
+
+--- !ELF
+FileHeader:
+  Class:   ELFCLASS64
+  Data:    ELFDATA2LSB
+  Type:    ET_DYN
+  Machine: EM_RISCV