[PATCH] D18248: [ELF] - Set the sh_entsize for mergable sections

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 18 02:56:03 PDT 2016


grimar added inline comments.

================
Comment at: lld/trunk/ELF/OutputSections.cpp:1223
@@ -1222,2 +1222,3 @@
   uintX_t EntSize = S->getSectionHdr()->sh_entsize;
+  this->Header.sh_entsize = EntSize;
 
----------------
ruiu wrote:
> Setting a header value from addSection would be a bit confusing because it runs multiple times. I'd do that in finalize().
Am I missing something, but we dont have access to sh_entsize from finalize. I mean we dont store a list of added sections to take it there.


Repository:
  rL LLVM

http://reviews.llvm.org/D18248





More information about the llvm-commits mailing list