[cfe-commits] [patch] --hash-style linker option for Mips targets

[Chandler Carruth]

On Thu, Dec 8, 2011 at 8:24 PM, Simon Atanasyan <satanasyan at mips.com> wrote:

> Please commit because I do not have the commit access.
>

Done in r146243.

Any chance you could work up a test case that could be added to
test/Driver/linux-ld.c that would exercise this new functionality? I could
try, but I worry I'd not capture it correctly as I don't have any knowledge
of MIPS.


>
> On Thu, Dec 8, 2011 at 2:30 PM, Chandler Carruth <chandlerc at google.com>
> wrote:
> > This is fine, please commit. If you don't have commit access, I'll commit
> > tomorrow. Thanks for fixing!
> >
> > On Thu, Dec 8, 2011 at 2:14 AM, Atanasyan, Simon <satanasyan at mips.com>
> > wrote:
> >>
> >> Clang driver passes --hash-style=gnu / --hash-style=both options to the
> >> linker when the driver runs on some Linux distributions. If a target
> >> architecture is Mips, the linker shows the following error:
> >>
> >> mips-linux-gnu-ld: .gnu.hash is incompatible with the MIPS ABI
> >>
> >> That is because .gnu.hash and the MIPS ABI require .dynsym to be sorted
> in
> >> different ways. .gnu.hash needs symbols to be grouped by hash code
> whereas
> >> the MIPS ABI requires a mapping between the GOT and the symbol table.
> Here
> >> is the patch to fix this problem.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20111208/3e132d47/attachment.html>