<div dir="ltr">If the tool does not support that and if it would take time to add the feature, it's OK to check in binary files for now.<div><br></div><div>But your patch does not contain the .so file because diff does not compare binary files. Please send me the file so that I can submit.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 22, 2014 at 11:59 AM, Rafael Auler <span dir="ltr"><<a href="mailto:rafaelauler@gmail.com" target="_blank">rafaelauler@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rui,<br>
<br>
I updated the patch to use obj2yaml and yaml2obj to avoid binary files. While I was able to do this to the simple defobj.o file, I was not able to convert the YAML form of libundef2.so to a .so file. The error I got was:<br>
<br>
error: Unknown section referenced: '.dynsym' at YAML section '.hash'.<br>
<br>
Then I investigated my libundef2.so yaml representation and noticed that it was really missing .dymsym. I looked obj2yaml's source code and found this:<br>
<br>
  // Dump sections<br>
  for (const Elf_Shdr &Sec : Obj.sections()) {<br>
    switch (Sec.sh_type) {<br>
    case ELF::SHT_NULL:<br>
    case ELF::SHT_SYMTAB:<br>
    case ELF::SHT_DYNSYM:<br>
    case ELF::SHT_STRTAB:<br>
      // Do not dump these sections.<br>
<br>
...which suggested that the tool currently does not support dumping the dynamic symbol table.<br>
<br>
Therefore, my new patch still depends on uploading the libundef2.so binary file. Is there any problem?<br>
<br>
Best regards,<br>
Rafael Auler<br>
<span class=""><br>
<a href="http://reviews.llvm.org/D5424" target="_blank">http://reviews.llvm.org/D5424</a><br>
<br>
Files:<br>
  include/lld/ReaderWriter/ELFLinkingContext.h<br>
  lib/ReaderWriter/ELF/ELFLinkingContext.cpp<br>
  lib/ReaderWriter/ELF/OutputELFWriter.h<br>
</span>  test/elf/Inputs/libundef2.so<br>
  test/elf/undef-from-dso-to-main.test<br>
</blockquote></div><br></div>