Re: [Patch v2]: Fix ld pr11138 FAILures on mips*.
- To: David Daney <david.daney@cavium.com>
- Cc: binutils <binutils@sourceware.org>, Richard Sandiford <rdsandiford@googlemail.com>, Manuel Lauss <manuel.lauss@googlemail.com>, Debian MIPS <debian-mips@lists.debian.org>, linux-mips <linux-mips@linux-mips.org>
- Subject: Re: [Patch v2]: Fix ld pr11138 FAILures on mips*.
- From: Alan Modra <amodra@gmail.com>
- Date: Sat, 10 Dec 2011 11:09:28 +1030
- Message-id: <[🔎] 20111210003928.GC2461@bubble.grove.modra.org>
- Mail-followup-to: David Daney <david.daney@cavium.com>, binutils <binutils@sourceware.org>, Richard Sandiford <rdsandiford@googlemail.com>, Manuel Lauss <manuel.lauss@googlemail.com>, Debian MIPS <debian-mips@lists.debian.org>, linux-mips <linux-mips@linux-mips.org>
- In-reply-to: <[🔎] 4EE27012.5030508@cavium.com>
- References: <[🔎] 4EE27012.5030508@cavium.com>
On Fri, Dec 09, 2011 at 12:31:14PM -0800, David Daney wrote:
> * /elfxx-mips.c (mips_elf_link_hash_table.rld_value): Remove.
> (mips_elf_link_hash_table.rld_symbol): New field;
> (MIPS_ELF_RLD_MAP_SIZE): New macro.
> (_bfd_mips_elf_add_symbol_hook): Remember __rld_obj_head symbol
> in rld_symbol.
> (_bfd_mips_elf_create_dynamic_sections): Remember __rld_map symbol
> in rld_symbol.
> (_bfd_mips_elf_size_dynamic_sections): Set correct size for .rld_map.
> (_bfd_mips_elf_finish_dynamic_symbol): Remove .rld_map handling.
> (_bfd_mips_elf_finish_dynamic_sections): Use rld_symbol to
> calculate DT_MIPS_RLD_MAP value.
> (_bfd_mips_elf_link_hash_table_create): Initialize rld_symbol,
> quit initializing rld_value.
OK. Remove stray / in ChangeLog entry
> + s->size += MIPS_ELF_RLD_MAP_SIZE(output_bfd);
Fix formatting here.
> + dyn.d_un.d_ptr = s->output_section->vma + s->output_offset
> + + h->root.u.def.value;
And it's nice to write code that emacs auto-indent won't change, so
add parentheses
dyn.d_un.d_ptr = (s->output_section->vma + s->output_offset
+ h->root.u.def.value);
--
Alan Modra
Australia Development Lab, IBM
Reply to: