[arm] Don't "adjust" EABI relocations

[Paul Brook]

On Tuesday 30 November 2004 17:26, Richard Earnshaw wrote:
> On Tue, 2004-11-30 at 17:28, Nick Clifton wrote:
> > Hi Paul,
> >
> > >>Actually other, non-GNU, toolchains do that as well.  So supporting the
> > >>-p option for inter-linking with them is very helpful.
> > >
> > > But are there any other toolchains that don't need -p? We're suggesting
> > > removing the option altogether and making -p a nop.
> >
> > Oh - I see.  Sorry I had that the wrong way round.  Making the -p
> > behaviour enabled by default and ignoring the option if it is specified
> > on the command line is OK by me.
> >
> > Cheers
> >    Nick
>
> I think the option is the default.  At least, the arm-linux
> configuration of gcc is the only one that passes it, but all other
> platforms have the same behaviour.
>
> So I'm not really sure what the option is doing.

It's only the default for files with a nonzero EI_OSABI. This includes all gnu 
targets, but not all toolchains from other vendors.

The attached patch removes the code, and makes the commandline options do 
nothing.
Retested and applied.

Paul

2004-11-30  Paul Brook  <paul@codesourcery.com>

bfd/
 * elf32-arm.c (struct elf32_arm_link_hash_table): Remove
 no_pipeline_knowledge
 (elf32_arm_link_hash_table_create): Ditto.
 (bfd_elf32_arm_process_before_allocation): Ditto.
 (elf32_arm_final_link_relocate): Ditto.  Remove oabi relocation
 handling.
 * bfd-in.h (bfd_elf32_arm_process_before_allocation): Update
 prototype.
 * bfd-in2.h: Regenerate.
ld/
 * emultempl/armelf.em: Ignore -p and -no-pipeline-knowledge.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.binutils_osabi
Type: text/x-diff
Size: 8441 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20041130/34b18dd1/attachment.bin>