[PATCH] D18135: ELF: Move initSymbols to Symbols.cpp.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 10:41:23 PDT 2016


ruiu added a comment.

Sorry, I replied to a different thread.

So, it's ecause that Symbol.cpp needs to initialize symbols for each ELF type is an internal detail of the file. initSymbol, or in general, init<filename> is the initializer function for a file. Callers of the initializer should not take care of the details whether the function works on ELFT-specialized objects or not.


http://reviews.llvm.org/D18135





More information about the llvm-commits mailing list