[Bug 239743] Mk/bsd.port.mk: don't switch to ld.bfd on powerpc64 with clang

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=239743

Jan Beich <jbeich at FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #206395|0                           |1
        is obsolete|                            |
                 CC|                            |jbeich at FreeBSD.org

--- Comment #13 from Jan Beich <jbeich at FreeBSD.org> ---
Created attachment 206411
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=206411&action=edit
patch

Antoine, does this work for you? It worked for pkubaj at . I'm not sure if my
commit message/inline comment are correct, though.

- Only one extra conditional (easy to understand)
- Comment why there (easy to refactor)
- Drop OSVERSION to avoid redundancy with ld == lld conditional
- Add -B${LOCALBASE}/bin because USE_BINUTILS alone doesn't

(In reply to Piotr Kubaj from comment #4)
> Regarding why powerpc64 needs special treatment:
> when it switches to clang, ld.bfd will still be installed.
> The reason is that lld currently doesn't support linking powerpc (32-bit) binaries.
> We will use lld for 64-bit binaries, however.

Do you mean -m64 uses ELFv2 while -m32 uses ELFv1? If so either review D20261
or review D20383 needs to make Clang default to BFD linker when -m32 is passed
e.g., via getDefaultLinker().

-- 
You are receiving this mail because:
You are on the CC list for the bug.