<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Apr 30, 2016 at 4:15 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 30 April 2016 at 19:01, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> On Sat, Apr 30, 2016 at 12:34 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> > -  void parse();<br>
>> > -<br>
>> > -  llvm::MutableArrayRef<LazyObject> getLazySymbols() { return<br>
>> > LazySymbols; }<br>
>> > +  void parse(SymbolTableBase *Symtab);<br>
>> ><br>
>> > ----------------<br>
>> > Can we merge SymbolTableBase with SymbolTable<ELFT> if you template<br>
>> > this, ArchiveFile::parse and BitcodeFile::parse functions?<br>
>><br>
>> I assume these vectors make it hard:<br>
>><br>
>>  std::vector<std::unique_ptr<ObjectFile<ELFT>>> ObjectFiles;<br>
>>   std::vector<std::unique_ptr<SharedFile<ELFT>>> SharedFiles;<br>
><br>
><br>
> I was suggesting defining only SymbolTable<ELFT> and remove SymbolTableBase.<br>
<br>
</div></div>But then having a global pointer to it is somewhat inconvenient.<br></blockquote><div><br></div><div>Why? We can do like we do for Out, no? </div></div></div></div>