diff options
author | Alan Modra <amodra@gmail.com> | 2023-05-19 17:40:30 +0930 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2023-05-20 10:49:19 +0930 |
commit | 880853ed941dc1154e3bb2bd44ddd10b84dfecff (patch) | |
tree | f862234c33c3ae6738512e6ca642f83e26087e4a /gdb | |
parent | b196c8b3b6027714f50f75c1278843ee0e38d6de (diff) | |
download | binutils-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