<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 24, 2015 at 9:08 PM, 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"><span class="">On Thu, Sep 24, 2015 at 8:55 PM, Shankar Easwaran via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> shankare added a subscriber: shankare.<br>
><br>
> ================<br>
> Comment at: ELF/Writer.cpp:339-340<br>
> @@ -338,4 +338,4 @@<br>
>          ErrorOr<StringRef> SymName = Sym.getName(File.getStringTable());<br>
> -        if (SymName && SymTabSec.shouldKeepInSymtab(*SymName))<br>
> +        if (SymName && shouldKeepInSymtab(*SymName))<br>
>            SymTabSec.addSymbol(*SymName, true);<br>
>        }<br>
> ----------------<br>
> what if discard all and discard none is used ? Do we plan to print any warning of what option was chosen ?<br>
><br>
<br>
</span>discard-all "wins". This is also what ld.bfd does, FWIW, and it also<br>
doesn't emit any warning about it.<br>
I didn't have a warning in mind but if there's a request + consensus<br>
I'm not strongly opposed to adding it (at some point).<br></blockquote><div><br></div><div>I agree that we don't have to print out an error message at least now. But I'm thinking that if these options are exclusive, shouldn't we define enums for {none,locals,all} instead of having three separate flags in Config?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><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></div><br></div></div>