Date: Sat, 10 Aug 2019 04:48:33 +0000 From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 239743] Mk/bsd.port.mk: don't switch to ld.bfd on powerpc64 with clang Message-ID: <bug-239743-7788-2J2qB4xV6V@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-239743-7788@https.bugs.freebsd.org/bugzilla/> References: <bug-239743-7788@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D239743 Jan Beich <jbeich@FreeBSD.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #206395|0 |1 is obsolete| | CC| |jbeich@FreeBSD.org --- Comment #13 from Jan Beich <jbeich@FreeBSD.org> --- Created attachment 206411 --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=3D206411&action= =3Dedit patch Antoine, does this work for you? It worked for pkubaj@. 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 =3D=3D 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 D202= 61 or review D20383 needs to make Clang default to BFD linker when -m32 is pas= sed e.g., via getDefaultLinker(). --=20 You are receiving this mail because: You are on the CC list for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-239743-7788-2J2qB4xV6V>