[PATCH] D25826: [ELF] Show error location for 'undefined symbol' errors

[Eugene Leviant via llvm-commits]

evgeny777 added inline comments.


================
Comment at: ELF/InputFiles.cpp:407
+DefinedRegular<ELFT> *
+elf::ObjectFile<ELFT>::getEnclosingSymbol(InputSectionBase<ELFT> *S,
+                                          uintX_t Offset) {
----------------
ruiu wrote:
> This function can be implemented as a non-member function, no? Non-member functions are preferred over member functions if they access only public members.
No, it can't. It iterates SymbolBodies, which is member variable.


https://reviews.llvm.org/D25826