[PATCH] D28773: [ELF] - Avoid reusing DynsymIndex for -r

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 21 16:30:12 PST 2017


ruiu added inline comments.


================
Comment at: ELF/InputSection.cpp:233
 
+template <class ELFT> static uint32_t getSymbolIndex(SymbolBody &Body) {
+  if (Body.IsLocal) {
----------------
This function seems too heavy to run for each relocation. It is probably not ok to do this much here.


https://reviews.llvm.org/D28773





More information about the llvm-commits mailing list