[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