[lld] r248133 - [ELF2] Implement support for -discard-locals.

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 19:00:40 PDT 2015


On Mon, Sep 21, 2015 at 6:15 AM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
>> +        if (Config->DiscardLocals && SymName->startswith(".L"))
>> +          continue;
>>          ESym->st_name = (SymName) ? StrTabSec.getFileOff(*SymName) : 0;
>
> It is probably a good idea to make this a static helper
> (shouldIncludeInStaticSymtab).
>
> If you are interested in this area (controlling what goes in the
> static symtab), I think what is missing is
>
> * Add a --discard-none.

http://reviews.llvm.org/D13083

I think with this all the options for symbols are implemented. We can
discuss about defaults at a later time I think (my guess is that the
first version of the linker is gonna be chubby).


> On this test, it probably makes sense to include the ASCII rendering of .strtab.
>

Done in r248361.

Thanks!

--
Davide

"There are no solved problems; there are only problems that are more
or less solved" -- Henri Poincare


More information about the llvm-commits mailing list