<div dir="ltr">Ah, so it is discussed here. I'd love to see -save-temps being used instead of introducing a new flag.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 21, 2016 at 11:39 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I like Peter's idea: Lets keep local names only when using -save-temps.<div><br></div><div>Cheers,</div><div>Rafael</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On 21 April 2016 at 14:32, Teresa Johnson <span dir="ltr"><<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Thu, Apr 21, 2016 at 11:18 AM, Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On Thu, Apr 21, 2016 at 10:57 AM, Peter Collingbourne <<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>> wrote:<br>
><br>
><br>
> On Thu, Apr 21, 2016 at 10:46 AM, Davide Italiano via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>><br>
>> Author: davide<br>
>> Date: Thu Apr 21 12:46:38 2016<br>
>> New Revision: 267020<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=267020&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=267020&view=rev</a><br>
>> Log:<br>
>> [LTO] Discard names for values that are not global by default.<br>
>><br>
>> Rafael reported on the mailing list that this reduces peak memory<br>
>> usage while linking llvm-as by 15%. It makes sense to make it<br>
>> the default, and introduce an inverse knob -lto-no-discard-value-names<br>
>> for those who want to restore the old behavior.<br>
>><br>
>> Modified:<br>
>>     lld/trunk/ELF/Driver.cpp<br>
>>     lld/trunk/ELF/Options.td<br>
>>     lld/trunk/test/ELF/lto/discard-value-names.ll<br>
>>     lld/trunk/test/ELF/lto/type-merge2.ll<br>
>><br>
>> Modified: lld/trunk/ELF/Driver.cpp<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Driver.cpp?rev=267020&r1=267019&r2=267020&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Driver.cpp?rev=267020&r1=267019&r2=267020&view=diff</a><br>
>><br>
>> ==============================================================================<br>
>> --- lld/trunk/ELF/Driver.cpp (original)<br>
>> +++ lld/trunk/ELF/Driver.cpp Thu Apr 21 12:46:38 2016<br>
>> @@ -289,7 +289,7 @@ void LinkerDriver::readConfigs(opt::Inpu<br>
>>    Config->Demangle = !Args.hasArg(OPT_no_demangle);<br>
>>    Config->DisableVerify = Args.hasArg(OPT_disable_verify);<br>
>>    Config->DiscardAll = Args.hasArg(OPT_discard_all);<br>
>> -  Config->DiscardValueNames = Args.hasArg(OPT_lto_discard_value_names);<br>
>> +  Config->DiscardValueNames =<br>
>> !Args.hasArg(OPT_lto_no_discard_value_names);<br>
>>    Config->DiscardLocals = Args.hasArg(OPT_discard_locals);<br>
>>    Config->DiscardNone = Args.hasArg(OPT_discard_none);<br>
>>    Config->EhFrameHdr = Args.hasArg(OPT_eh_frame_hdr);<br>
>><br>
>> Modified: lld/trunk/ELF/Options.td<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=267020&r1=267019&r2=267020&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=267020&r1=267019&r2=267020&view=diff</a><br>
>><br>
>> ==============================================================================<br>
>> --- lld/trunk/ELF/Options.td (original)<br>
>> +++ lld/trunk/ELF/Options.td Thu Apr 21 12:46:38 2016<br>
>> @@ -238,7 +238,7 @@ def G : Separate<["-"], "G">;<br>
>>  def alias_version_script_version_script : Joined<["--"],<br>
>> "version-script=">, Alias<version_script>;<br>
>><br>
>>  // LTO-related options.<br>
>> -def lto_discard_value_names : Flag<["-"], "lto-discard-value-names">;<br>
>> +def lto_no_discard_value_names : Flag<["-"],<br>
>> "lto-no-discard-value-names">;<br>
><br>
><br>
> We probably don't want to name flags "-l(something)", as that would conflict<br>
> with the -l flag. Can you please rename this "--lto-..."?<br>
><br>
> (That is, if we want a flag for this at all. Have you considered gating this<br>
> behavior behind -save-temps instead?)<br>
><br>
<br>
</div></div>I'm fine with one less flag (and using -save-temps), but I'm cc:ing<br>
the other people involved to hear if they agree.<br></blockquote><div><br></div></div></div><div>Interesting idea. When I added this to gold-plugin yesterday morning I also had to add a no- variant so that certain tests would keep working. If lld goes the route of using -save-temps we should consider doing the same for gold-plugin for consistency and flag reduction. </div><div><br></div><div>Teresa</div><span><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><font color="#888888"><br>
<br>
--<br>
Davide<br>
<br>
"There are no solved problems; there are only problems that are more<br>
or less solved" -- Henri Poincare<br>
</font></span></blockquote></span></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> <a href="tel:408-460-2413" value="+14084602413" target="_blank">408-460-2413</a></td></tr></tbody></table></span></div>
</font></span></div></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>