[PATCH] D65278: [llvm-objcopy] - Stop using Inputs/alloc-symtab.o.

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: test/tools/llvm-objcopy/ELF/binary-out-error.test:2
+# RUN: yaml2obj %s -o %t
+# RUN: not llvm-objcopy -O binary %t %t2 2>&1 >/dev/null | FileCheck %s --check-prefix=SYMTAB
 # SYMTAB: cannot write symbol table '.symtab' out to binary
----------------
jhenderson wrote:
> MaskRay wrote:
> > `not llvm-objcopy -O binary %t - 2>&1 | FileCheck %s --check-prefix=SYMTAB`
> > 
> You can actually get rid of the --check-prefix entirely, since this is the only check in the file.
Some other "out to binary" warnings have not been tested. We can add them later, so `--check-prefix=SYMTAB` can be kept to make future diff smaller.


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

https://reviews.llvm.org/D65278