From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Lance Taylor To: nickc@cygnus.com Cc: binutils@sourceware.cygnus.com Subject: Re: Patch to change ARM register name set Date: Thu, 01 Jul 1999 00:00:00 -0000 Message-id: <19990615163347.1080.qmail@daffy.airs.com> References: <199906150957.KAA18137@pathia.cygnus.co.uk> <199906150957.KAA18137@pathia.cygnus.co.uk> X-SW-Source: 1999-q2/msg00259.html Date: Tue, 15 Jun 1999 10:57:16 +0100 From: Nick Clifton : However, adding a new disassembler2 function does not make sense to : me. Adding new functions because you have a new parameter is an : approach that leads to ABI complexity. It's part of what makes the : Win32 ABI so difficult to work with. Don't tread that path. Instead, : just change the existing function. OK - that is actually what I wanted to do in the first place, but I did not know if there were lots of utilities out there that use the current disassembler() function, so I did not want to change it. I doubt anything uses disassembler other than the binutils and gdb. : In objdump.c, I think target_data is a poor choice of names. This is : an option to the disassembler. objdump does many things other than : disassemble, so using a word like `target_data' to describe something : which is specific to the disassembler is misleading. Something like : `disassembler_options' makes more sense to me. This should be changed : in the suggested documentation as well. OK - although I had hoped that this might turn into a more generic type of command line switch where the text could be passed to other components of objdump, not just its disassembler. That's an interesting idea. For most of what objdump does, though, there is no code which could interpret the option. It only makes sense to have a generic option like that in cases where objdump invokes target specific code. The only case which comes to mind other than disassembly is the -p option to print private data. I could imagine cases where it would be useful to pass an option to -p, but I think that generally it would not make sense. I don't really want to go down the path of passing a lot of options to -p to control what it prints. However, perhaps there is something I am not thinking of. Ian