[PATCH] D18554: [ELF, PR27016] - Do not keep undefined locals in .symtab

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 08:58:01 PDT 2016


grimar added a comment.

In http://reviews.llvm.org/D18554#385680, @emaste wrote:

> Should be a continue in copyLocalSymbols instead?


I think both ways are ok, but I prefer mine. That is exactly what we do - just do not put them in symtab and for me It is ok check it in shouldKeepInSymtab.


http://reviews.llvm.org/D18554





More information about the llvm-commits mailing list