Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

Replace bfd_hostptr_t with uintptr_t

Browse Source 
bfd_hostptr_t is defined as a type large enough to hold either a long
or a pointer.  It mostly appears in the coff backend code in casts.
include/coff/internal.h struct internal_syment and union
internal_auxent have the only uses in data structures, where
comparison with include/coff/external.h and other code reveals that
the type only needs to be large enough for a 32-bit integer or a
pointer.  That should mean replacing with uintptr_t is OK.
This commit is contained in:
Alan Modra
2022-05-27 15:28:34 +09:30
parent 65d13793d9
commit 6015985895
14 changed files with 50 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
include/coff/internal.h
View File

@@ -469,8 +469,8 @@ struct internal_syment
char _n_name[SYMNMLEN] ATTRIBUTE_NONSTRING; /* old COFF version */
struct
{
bfd_hostptr_t _n_zeroes; /* new == 0 */
bfd_hostptr_t _n_offset; /* offset into string table */
uintptr_t _n_zeroes; /* new == 0 */
uintptr_t _n_offset; /* offset into string table */
} _n_n;
char *_n_nptr[2]; /* allows for overlaying */
} _n;
@@ -595,11 +595,11 @@ union internal_auxent
char x_fname[20];
struct
{
/* PR 28630: We use bfd_hostptr_t because these fields may be
/* PR 28630: We use uintptr_t because these fields may be
used to hold pointers. We assume that this type is at least
as big as the long type. */
bfd_hostptr_t x_zeroes;
bfd_hostptr_t x_offset;
32 bits. */
uintptr_t x_zeroes;
uintptr_t x_offset;
} x_n;
} x_n;