diff options
| author | Dave Anglin <dave.anglin@nrc.ca> | 2003-01-08 02:19:23 +0000 |
|---|---|---|
| committer | Dave Anglin <dave.anglin@nrc.ca> | 2003-01-08 02:19:23 +0000 |
| commit | df50c9cac4a8902e6e2a3a6adf7bbcca8520c747 (patch) | |
| tree | 9428b849a336f6b0662b5f63d746afd0fa85b9cd | |
| parent | ef39abc298bc94035e86a82dc2d2789529bbda51 (diff) | |
| download | gdb-df50c9cac4a8902e6e2a3a6adf7bbcca8520c747.tar.gz | |
* elf32-hppa.c (final_link_relocate): For all DP relative relocations,
adjust addil instructions if the symbol has no section.
| -rw-r--r-- | bfd/ChangeLog | 5 | ||||
| -rw-r--r-- | bfd/elf32-hppa.c | 13 |
2 files changed, 11 insertions, 7 deletions
diff --git a/bfd/ChangeLog b/bfd/ChangeLog index fde02c86565..81f78a0010a 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2003-01-07 John David Anglin <dave.anglin@nrc.gc.ca> + + * elf32-hppa.c (final_link_relocate): For all DP relative relocations, + adjust addil instructions if the symbol has no section. + 2003-01-07 DJ Delorie <dj@redhat.com> * elf32-xstormy16.c (xstormy16_elf_howto_table): Make REL_12 not diff --git a/bfd/elf32-hppa.c b/bfd/elf32-hppa.c index f8c88b15583..2646f3dc4ed 100644 --- a/bfd/elf32-hppa.c +++ b/bfd/elf32-hppa.c @@ -3416,15 +3416,14 @@ final_link_relocate (input_section, contents, rel, value, htab, sym_sec, h) case R_PARISC_DPREL14R: case R_PARISC_DPREL14F: /* For all the DP relative relocations, we need to examine the symbol's - section. If it's a code section, then "data pointer relative" makes - no sense. In that case we don't adjust the "value", and for 21 bit - addil instructions, we change the source addend register from %dp to - %r0. This situation commonly arises when a variable's "constness" + section. If it has no section or if it's a code section, then + "data pointer relative" makes no sense. In that case we don't + adjust the "value", and for 21 bit addil instructions, we change the + source addend register from %dp to %r0. This situation commonly + arises for undefined weak symbols and when a variable's "constness" is declared differently from the way the variable is defined. For instance: "extern int foo" with foo defined as "const int foo". */ - if (sym_sec == NULL) - break; - if ((sym_sec->flags & SEC_CODE) != 0) + if (sym_sec == NULL || (sym_sec->flags & SEC_CODE) != 0) { if ((insn & ((0x3f << 26) | (0x1f << 21))) == (((int) OP_ADDIL << 26) | (27 << 21))) |