[all-commits] [llvm/llvm-project] 0ad18b: [llvm-objdump] Suppress spurious warnings when par...
Michael Trent via All-commits
all-commits at lists.llvm.org
Mon Feb 3 10:59:41 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: 0ad18bf37b2d4f6a619f2ebdd6cfbe0ac617a2e5
https://github.com/llvm/llvm-project/commit/0ad18bf37b2d4f6a619f2ebdd6cfbe0ac617a2e5
Author: Michael Trent <mtrent at apple.com>
Date: 2020-02-03 (Mon, 03 Feb 2020)
Changed paths:
M llvm/test/tools/llvm-objdump/X86/macho-cstring-dump.test
M llvm/tools/llvm-objdump/MachODump.cpp
M llvm/tools/llvm-objdump/llvm-objdump.cpp
Log Message:
-----------
[llvm-objdump] Suppress spurious warnings when parsing Mach-O binaries.
Summary:
llvm-objdump started warning when asked to disassemble a section that
isn't present in the input files, in Yuanfang Chen's change:
d16c162c9453db855503134fe29ae4a3c0bec936. The problem is that the
logic was restricted only to the generic llvm-objdump parser, not to the
Mach-O-specific parser used for Apple toolchain compatibility. The
solution is to log section names from the Mach-O parser.
The macho-cstring-dump.test has been updated to fail if it encounters
this new warning in the future.
Reviewers: pete, ab, lhames, jhenderson, grimar, MaskRay, ychen
Reviewed By: jhenderson, grimar
Subscribers: rupprecht, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73586
More information about the All-commits
mailing list