<div dir="ltr">That's the Fuchsia CMake cache file which is used to build Fuchsia toolchain, it's not needed there anymore because Fuchsia Clang driver now handles this. I haven't touched Clang's CMakeLists.txt which defines the ENABLE_X86_RELAX_RELOCATIONS option.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Nov 21, 2017 at 3:14 PM Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I am probably missing something, but the patch has<br>
<br>
-# This is a "Does your linker support it?" option that only applies<br>
-# to x86-64 ELF targets.  All Fuchsia target linkers do support it.<br>
-# For x86-64 Linux, it's supported by LLD and by GNU linkers since<br>
-# binutils 2.27, so one can hope that all Linux hosts in use handle it.<br>
-# Ideally this would be settable as a per-target option.<br>
-set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")<br>
<br>
And I don't see what that is replaced with.<br>
<br>
Cheers,<br>
Rafael<br>
<br>
Petr Hosek <<a href="mailto:phosek@google.com" target="_blank">phosek@google.com</a>> writes:<br>
<br>
> I kept the CMake option, so by default the driver will use the value set<br>
> through CMake as before but individual targets can now set their platform<br>
> default.<br>
><br>
> On Tue, Nov 21, 2017 at 1:30 PM Rafael Avila de Espindola <<br>
> <a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br>
><br>
>> Petr Hosek via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> writes:<br>
>><br>
>><br>
>> > -# This is a "Does your linker support it?" option that only applies<br>
>> > -# to x86-64 ELF targets.  All Fuchsia target linkers do support it.<br>
>> > -# For x86-64 Linux, it's supported by LLD and by GNU linkers since<br>
>> > -# binutils 2.27, so one can hope that all Linux hosts in use handle it.<br>
>> > -# Ideally this would be settable as a per-target option.<br>
>> > -set(ENABLE_X86_RELAX_RELOCATIONS ON CACHE BOOL "")<br>
>> > -<br>
>><br>
>> We still have to be able to set it via cmake.<br>
>><br>
>> Cheers,<br>
>> Rafeal<br>
>><br>
</blockquote></div>