This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
RE: [PATCH ARC 2]: Local symbols relocations cleanup
- From: Claudiu Zissulescu <Claudiu dot Zissulescu at synopsys dot com>
- To: Nick Clifton <nickc at redhat dot com>, Cupertino Miranda <Cupertino dot Miranda at synopsys dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Cc: Francois Bedard <Francois dot Bedard at synopsys dot com>
- Date: Mon, 29 Feb 2016 15:54:28 +0000
- Subject: RE: [PATCH ARC 2]: Local symbols relocations cleanup
- Authentication-results: sourceware.org; auth=none
- References: <EB86EB452ADE4B44B294F7149B8A27770218DF5C at DE02WEMBXB dot internal dot synopsys dot com> <56D07D11 dot 4030407 at redhat dot com>
Hi,
I've committed the patch. However, the ld-elf/merge will start failing. As far as I understood, this error will be fixed by the third patch in the queue.
Thanks,
Claudiu
> -----Original Message-----
> From: Nick Clifton [mailto:nickc@redhat.com]
> Sent: Friday, February 26, 2016 5:28 PM
> To: Cupertino Miranda; binutils@sourceware.org
> Cc: Claudiu Zissulescu; Francois Bedard
> Subject: Re: [PATCH ARC 2]: Local symbols relocations cleanup
>
> Hi Cupertino,
>
> > bfd/ChangeLog:
> >
> > Cupertino Miranda <cmiranda@synopsys.com>
> > * elf32-arc.c (elf_arc_relocate_section): Added rules to fix the
> > relocation addend when sections get merged.
> >
> >
> > gas/ChangeLog:
> >
> > Cupertino Miranda <cmiranda@synopsys.com>
> > Claudiu Zissulescu <Claudiu.Zissulescu@synopsys.com>
> > * config/tc-arc.c (arc_extra_reloc): Change size to 0.
> > (tc_arc_fix_adjustable): Changed default return value to 1.
> >
> > gas/ChangeLog:
> >
> > Cupertino Miranda <cmiranda@synopsys.com>
> > * testsuite/gas/arc/j.d: Updated expected symbol
> > * testsuite/gas/arc/jl.d: Likewise
> > * testsuite/gas/arc/relax-avoid1.d: Likewise
> > * testsuite/gas/arc/st.d: Likewise
> >
> > ld/ChangeLog:
> >
> > Cupertino Miranda <cmiranda@synopsys.com>
> > * testsuite/ld-elf/merge.d: Removed xfail for ARC.
> > * testsuite/ld-elf/merge2.d: Likewise.
> > * testsuite/ld-elf/merge3.d: Likewise.
> >
>
> Approved - please apply.
>
> Cheers
> Nick