[PATCH] D19566: ELF: Merge UndefinedBitcode and UndefinedElf.

[Peter Collingbourne via llvm-commits]

pcc added inline comments.

================
Comment at: ELF/InputFiles.cpp:320
@@ -320,1 +319,3 @@
+      return new (Alloc)
+          Undefined(Sym->st_name, Sym->st_other, Sym->getType(), Sym->st_size);
     return new (Alloc) DefinedRegular<ELFT>(*Sym, Sec);
----------------
ruiu wrote:
> This may be a silly question, but I don't remember why we need to save st_size for undefined symbols. Are we using that information?
The size shows up in the output symbol table, but I can't really think of a good reason why we need to keep it there. I'll change this patch to drop it unless @rafael can see a reason not to.


http://reviews.llvm.org/D19566