[PATCH] D35936: [ELF] - Move getSymbols() method to InputFile.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 06:30:10 PDT 2017


ruiu added a comment.

Overall, it doesn't seem like an improvement. You don't need to push too hard to remove templates. Use of templates is not necessarily bad.

That being said, if you think that this change could remove a lot of uses of templates, please do that in this patch to demonstrate it.


https://reviews.llvm.org/D35936





More information about the llvm-commits mailing list