[PATCH] [lld][ELF][Writer] Add dynamic symbol table.
Shankar Kalpathi Easwaran
shankarke at gmail.com
Thu Feb 21 14:43:24 PST 2013
SymbolTable::finalize already sets up the link properly. buildSectionHeaderTable only adds the sections to the section header table but the ordinals have been assigned way ahead in time.
I still dont see why shdr->sh_size = std::max(section->fileSize(), section->memSize()); is being done. filesize / memsize would differ only for bss sections.
I was thinking if we have Dynamic Symbol table as a seperate class, things specific to the dynamic symbol table could be added in future in a easier way (like dynhash section).
http://llvm-reviews.chandlerc.com/D442
More information about the llvm-commits
mailing list