[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