[PATCH] D73586: Suppress spurious warnings when parsing Mach-O bianries.

[George Rimar via Phabricator via llvm-commits]

grimar 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
----------------
I'd suggest using `--implicit-check-not="warning:"` FileCheck option instead. Testing the whole message is too brittle approach I think.


================
Comment at: llvm/tools/llvm-objdump/MachODump.cpp:1753
     for (const SectionRef &Section : O->sections()) {
-      StringRef SectName;
+      StringRef SectName = StringRef();
       Expected<StringRef> SecNameOrErr = Section.getName();
----------------
Why this line was changed?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73586