From: Jan Beulich Date: Fri, 7 Mar 2025 10:24:19 +0000 (+0100) Subject: objdump: permit disassembling multiple individual functions X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=cdd8492b05979842a1660e99862cd1680e52b28d;p=thirdparty%2Fbinutils-gdb.git objdump: permit disassembling multiple individual functions Compilers may split functions, e.g. into a "hot" and "cold" part, or they may emit special case instantiations (e.g. as a result of IPA). It can be helpful to be able to disassemble all of the parts or clones in one go. Permit using "--disassemble=" multiple times. --- diff --git a/binutils/doc/binutils.texi b/binutils/doc/binutils.texi index 08a854fd84c..73848e3b390 100644 --- a/binutils/doc/binutils.texi +++ b/binutils/doc/binutils.texi @@ -2432,11 +2432,11 @@ with ctags tool. Display the assembler mnemonics for the machine instructions from the input file. This option only disassembles those sections which are expected to contain instructions. If the optional @var{symbol} -argument is given, then display the assembler mnemonics starting at -@var{symbol}. If @var{symbol} is a function name then disassembly -will stop at the end of the function, otherwise it will stop when the -next symbol is encountered. If there are no matches for @var{symbol} -then nothing will be displayed. +argument is given (perhaps multiple times), then display the assembler +mnemonics starting at (all the) @var{symbol}. If @var{symbol} is a +function name then disassembly will stop at the end of the function, +otherwise it will stop when the next symbol is encountered. If there +are no matches for any @var{symbol} then nothing will be displayed. Note if the @option{--dwarf=follow-links} option is enabled then any symbol tables in linked debug info files will be read in and diff --git a/binutils/objdump.c b/binutils/objdump.c index 82397463c69..8fdbe03c4fb 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -132,7 +132,6 @@ static const char *prefix; /* --prefix */ static int prefix_strip; /* --prefix-strip */ static size_t prefix_length; static bool unwind_inlines; /* --inlines. */ -static const char * disasm_sym; /* Disassembly start symbol. */ static const char * source_comment; /* --source_comment. */ static bool visualize_jumps = false; /* --visualize-jumps. */ static bool color_output = false; /* --visualize-jumps=color. */ @@ -141,6 +140,12 @@ static int process_links = false; /* --process-links. */ static int show_all_symbols; /* --show-all-symbols. */ static bool decompressed_dumps = false; /* -Z, --decompress. */ +static struct symbol_entry + { + const char *name; + struct symbol_entry *next; + } *disasm_sym_list; /* Disassembly start symbol(s). */ + static enum color_selection { on_if_terminal_output, @@ -187,7 +192,7 @@ struct objdump_disasm_info bool require_sec; disassembler_ftype disassemble_fn; arelent *reloc; - const char *symbol; + struct symbol_entry *symbol_list; }; /* Architecture to disassemble for, or default if NULL. */ @@ -3898,7 +3903,7 @@ disassemble_section (bfd *abfd, asection *section, void *inf) the symbol we have just found. Then print the symbol and find the next symbol on. Repeat until we have disassembled the entire section or we have reached the end of the address range we are interested in. */ - do_print = paux->symbol == NULL; + do_print = paux->symbol_list == NULL; loop_until = stop_offset_reached; while (addr_offset < stop_offset) @@ -3938,9 +3943,9 @@ disassemble_section (bfd *abfd, asection *section, void *inf) pinfo->symtab_pos = -1; } - /* If we are only disassembling from a specific symbol, + /* If we are only disassembling from specific symbols, check to see if we should start or stop displaying. */ - if (sym && paux->symbol) + if (sym && paux->symbol_list) { if (do_print) { @@ -3977,8 +3982,16 @@ disassemble_section (bfd *abfd, asection *section, void *inf) } /* We are not currently printing. Check to see - if the current symbol matches the requested symbol. */ - if (streq (name, paux->symbol) + if the current symbol matches any of the requested symbols. */ + for (const struct symbol_entry *ent = paux->symbol_list; + ent; + ent = ent->next) + if (streq (name, ent->name)) + { + do_print = true; + break; + } + if (do_print && bfd_asymbol_value (sym) <= addr) { do_print = true; @@ -4175,7 +4188,7 @@ disassemble_data (bfd *abfd) disasm_info.dynrelbuf = NULL; disasm_info.dynrelcount = 0; aux.reloc = NULL; - aux.symbol = disasm_sym; + aux.symbol_list = disasm_sym_list; disasm_info.print_address_func = objdump_print_address; disasm_info.symbol_at_address_func = objdump_symbol_at_address; @@ -6229,7 +6242,14 @@ main (int argc, char **argv) case 'd': disassemble = true; seenflag = true; - disasm_sym = optarg; + if (optarg) + { + struct symbol_entry *sym = xmalloc (sizeof (*sym)); + + sym->name = optarg; + sym->next = disasm_sym_list; + disasm_sym_list = sym; + } break; case 'z': disassemble_zeroes = true; diff --git a/binutils/testsuite/binutils-all/multi-3.d b/binutils/testsuite/binutils-all/multi-3.d new file mode 100644 index 00000000000..bc6f74443a1 --- /dev/null +++ b/binutils/testsuite/binutils-all/multi-3.d @@ -0,0 +1,28 @@ +#name: objdump multiple --disassemble= +#source: multi1.s +#source: multi2.s +#ld: -r +#objdump: --disassemble=func --disassemble=func2 -wz +# ECOFF disassembly omits local symbols, for whatever reason. +#xfail: "alpha*-*-*ecoff" "alpha*-*-osf*" + +.*: +file format .* + +Disassembly of section .*: + +0+ : +[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#?[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#?[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* + +0+[0-1][0-9a-f] : +[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#... +0+[0-2][0-9a-f] : +[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#?[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#?[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* + +0+[0-3][0-9a-f] : +[ ]*[0-9a-f]+: [0-9a-f][0-9a-f].* +#pass diff --git a/binutils/testsuite/binutils-all/objdump.exp b/binutils/testsuite/binutils-all/objdump.exp index aa7ddda8700..76ee30f8ed9 100644 --- a/binutils/testsuite/binutils-all/objdump.exp +++ b/binutils/testsuite/binutils-all/objdump.exp @@ -263,6 +263,7 @@ if { ![istarget "*-*-aix*"] && [file normalize "$LD"] == [file normalize "$objdir/../ld/ld-new"]} then { run_dump_test multi-1 run_dump_test multi-2 + run_dump_test multi-3 } # Test objdump --disassemble=