aboutsummaryrefslogtreecommitdiff
path: root/gdb
diff options
context:
space:
mode:
authorAlan Modra <amodra@gmail.com>2023-05-19 17:40:30 +0930
committerAlan Modra <amodra@gmail.com>2023-05-20 10:49:19 +0930
commit880853ed941dc1154e3bb2bd44ddd10b84dfecff (patch)
treef862234c33c3ae6738512e6ca642f83e26087e4a /gdb
parentb196c8b3b6027714f50f75c1278843ee0e38d6de (diff)
downloadbinutils-880853ed941dc1154e3bb2bd44ddd10b84dfecff.zip
binutils-880853ed941dc1154e3bb2bd44ddd10b84dfecff.tar.gz
binutils-880853ed941dc1154e3bb2bd44ddd10b84dfecff.tar.bz2
tic54x set_arch_mach
The tic54x backend provides its own coff_set_arch_mach, but wants to use the standard coff_set_section_contents. BFD_JUMP_TABLE_WRITE defines both of these functions, so the code also provides a wrapper for coff_set_section_contents. This is all quite OK, but I was on a mission to remove unnecessary declarations in coffcode.h, and on deleting the one for coff_set_arch_mach ran into a warning about the function being unused. I could have kept that declaration with its ATTRIBUTE_UNUSED or written "static bool ATTRIBUTE_UNUSED" on the definition but the latter is not usual and looks odd to me. So I had a closer look at tic54x_set_arch_mach and decided the function is very likely wrong to allow bfd_arch_unknown. Thus the backend should be using the standard coff_set_arch_mach. * coff-tic54x.c: Use BFD_JUMP_TABLE_WRITE (coff) in target vecs. (tic54x_coff_set_arch_mach): Delete. (tic54x_set_section_contents): Delete. * coffcode.h: Delete unnecessary forward declarations.
Diffstat (limited to 'gdb')
0 files changed, 0 insertions, 0 deletions