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

Eugene Leviant via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 02:07:28 PDT 2016

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.


More information about the llvm-commits mailing list