[PATCH] D39831: [Driver] Make the use of relax relocations a per target option

Petr Hosek via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 21 14:15:05 PST 2017


I kept the CMake option, so by default the driver will use the value set
through CMake as before but individual targets can now set their platform
default.

On Tue, Nov 21, 2017 at 1:30 PM Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> Petr Hosek via Phabricator <reviews at reviews.llvm.org> writes:
>
>
> > -# This is a "Does your linker support it?" option that only applies
> > -# to x86-64 ELF targets.  All Fuchsia target linkers do support it.
> > -# For x86-64 Linux, it's supported by LLD and by GNU linkers since
> > -# binutils 2.27, so one can hope that all Linux hosts in use handle it.
> > -# Ideally this would be settable as a per-target option.
> > -set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")
> > -
>
> We still have to be able to set it via cmake.
>
> Cheers,
> Rafeal
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171121/9ec2d460/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4843 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171121/9ec2d460/attachment.bin>


More information about the cfe-commits mailing list