diff options
author | Alan Modra <amodra@gmail.com> | 2021-12-14 21:13:21 +1030 |
---|---|---|
committer | Alan Modra <amodra@gmail.com> | 2021-12-15 08:23:10 +1030 |
commit | eda0ddeb26a117660f3c0f54572053117641c2c3 (patch) | |
tree | 757d6033615d4a4bd0d45bad65083686b5c744e2 /gold/compressed_output.h | |
parent | 237f6eac1577223034a3e3436bd270428d097534 (diff) | |
download | gdb-eda0ddeb26a117660f3c0f54572053117641c2c3.zip gdb-eda0ddeb26a117660f3c0f54572053117641c2c3.tar.gz gdb-eda0ddeb26a117660f3c0f54572053117641c2c3.tar.bz2 |
XCOFF C_STSYM test failure on 32-bit host
This test was failing here and on another similar symbol:
[ 4](sec 1)(fl 0x00)(ty 0)(scl 143) (nx 0) 0x05d1745d11745d21 .bs
where correct output is
[ 4](sec 1)(fl 0x00)(ty 0)(scl 143) (nx 0) 0x000000000000000a .bs
The problem is caused by a 32-bit host pointer being sign-extended
when stored into a 64-bit bfd_vma, and then that value not being
trimmed back to 32 bits when used. The following belt-and-braces
patch fixes both the store and subsequent reads.
* coffcode.h (coff_slurp_symbol_table): Do not sign extend
when storing a host pointer to syment.n_value.
* coffgen.c (coff_get_symbol_info): Cast syment.n_value to a
bfd_hostptr_t before using in arithmetic.
(coff_print_symbol): Likewise.
Diffstat (limited to 'gold/compressed_output.h')
0 files changed, 0 insertions, 0 deletions