[PATCH] D29021: [ELF] - Stop handling local symbols in a special way.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 11:57:47 PST 2017


ruiu added a comment.

Or, if you find that adding a new field to Symbol class doesn't really hurt any metrics, you can simply add it. Or, even if it slows down the linker a little bit, you may still be able to convince that that is necessary. I believe adding a new field is the simplest and sufficient way to do what you want to do, but I think you want to stop to think and investigate a few different ways, and then make your conclusion (because all I said were my guesses and not conclusions in any sense.)


Repository:
  rL LLVM

https://reviews.llvm.org/D29021





More information about the llvm-commits mailing list