[PATCH] D73586: Suppress spurious warnings when parsing Mach-O bianries.
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 29 00:51:55 PST 2020
jhenderson added inline comments.
================
Comment at: llvm/test/tools/llvm-objdump/X86/macho-cstring-dump.test:5
+CHECK-NOT: warning: section '__TEXT,__cstring' mentioned in a -j/--section option, but not found in any input file
CHECK: Contents of (__TEXT,__cstring) section
----------------
grimar wrote:
> I'd suggest using `--implicit-check-not="warning:"` FileCheck option instead. Testing the whole message is too brittle approach I think.
+1. It also means you only need to specify the pattern in one place, rather than the current two.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73586/new/
https://reviews.llvm.org/D73586
More information about the llvm-commits
mailing list