diff options
| author | Daniel Thornburgh <mysterymath@gmail.com> | 2022-10-28 20:09:09 -0700 |
|---|---|---|
| committer | Daniel Thornburgh <mysterymath@gmail.com> | 2022-10-30 13:36:18 -0700 |
| commit | cc2457ca1bbdddefc09d72f889bc0389c90a831e (patch) | |
| tree | 49215eb0a90bc0b180e8e1714b90baff45eb5b44 /llvm/tools/llvm-objdump/llvm-objdump.cpp | |
| parent | 19ac69320846e6d741edddff422985e0e39492f2 (diff) | |
| download | llvm-cc2457ca1bbdddefc09d72f889bc0389c90a831e.zip llvm-cc2457ca1bbdddefc09d72f889bc0389c90a831e.tar.gz llvm-cc2457ca1bbdddefc09d72f889bc0389c90a831e.tar.bz2 | |
[llvm-objdump] Set --print-imm-hex by default.
This was previously attempted in 2016 by colinl's D18770, but LLD tests
were missed, which caused the change to be reverted.
Setting --print-imm-hex by default brings llvm-objdump's behavior closer
in line with objdump, and it makes it easier to read addresses and
alignment from the disassembly. It may make non-address immediates
harder to interpret, but it still seems the better default, barring more
context-sensitive base selection logic.
Differential Revision: https://reviews.llvm.org/D136972
Diffstat (limited to 'llvm/tools/llvm-objdump/llvm-objdump.cpp')
| -rw-r--r-- | llvm/tools/llvm-objdump/llvm-objdump.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/llvm-objdump/llvm-objdump.cpp b/llvm/tools/llvm-objdump/llvm-objdump.cpp index efa64ed..9dab4a7 100644 --- a/llvm/tools/llvm-objdump/llvm-objdump.cpp +++ b/llvm/tools/llvm-objdump/llvm-objdump.cpp @@ -3030,7 +3030,7 @@ static void parseObjdumpOptions(const llvm::opt::InputArgList &InputArgs) { RawClangAST = InputArgs.hasArg(OBJDUMP_raw_clang_ast); Relocations = InputArgs.hasArg(OBJDUMP_reloc); PrintImmHex = - InputArgs.hasFlag(OBJDUMP_print_imm_hex, OBJDUMP_no_print_imm_hex, false); + InputArgs.hasFlag(OBJDUMP_print_imm_hex, OBJDUMP_no_print_imm_hex, true); PrivateHeaders = InputArgs.hasArg(OBJDUMP_private_headers); FilterSections = InputArgs.getAllArgValues(OBJDUMP_section_EQ); SectionHeaders = InputArgs.hasArg(OBJDUMP_section_headers); |