This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: ld on irix revisited


> 
> Thiemo asked me to send an updated patch to either fix the problems
> or drop the section.  All in all, I think dropping it would be best.
> It is optional after all.
> 

I agree. We don't, and don't want to, use quickstart.

> So, the patch below gets rid of the most of the .msym code.
> I've kept bits in a few functions:
> 
>   - _bfd_mips_elf_section_from_shdr (interpret SHT_MIPS_MSYM sections)
>   - _bfd_mips_elf_fake_sections (set type and alignment of .msym)
>   - _bfd_mips_elf_finish_dynamic_sections (set value of DT_MIPS_MSYM)
>   - _bfd_mips_elf_final_write_processing (set .msym's sh_link)
> 
> I don't know whether all these are still needed are not, but there
> is similar code for other sections (like .liblist and .conflict)
> that bfd doesn't create itself.  Could someone more familiar with
> bfd double-check?

Yes. We need to still handle existing sections. The code that you keep
around to do that should be sufficient.

> 
> 	* elfxx-mips.c (mips_elf_link_hash_entry): Remove min_dyn_reloc_index.
> 	(bfd_mips_elf_swap_msym_in, bfd_mips_elf_swap_msym_out): Delete.
> 	(mips_elf_create_msym_section): Delete.
> 	(mips_elf_create_dynamic_relocation): Don't set min_dyn_reloc_index.
> 	(_bfd_mips_elf_copy_indirect_symbol): Likewise.
> 	(_bfd_mips_elf_create_dynamic_sections): Don't create .msym.
> 	(_bfd_mips_elf_size_dynamic_sections): Don't calculate its size.
> 	(_bfd_mips_elf_size_dynamic_sections): Don't add DT_MIPS_MSYM.
> 	(_bfd_mips_elf_finish_dynamic_symbol): Don't add symbols to .msym.
> 	(_bfd_mips_elf_finish_dynamic_sections): Likewise.

OK.

-eric

-- 
Eric Christopher <echristo@redhat.com>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]