[PATCH] D12944: [ELF2] Fill up local symbols field correctly

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 15:44:15 PDT 2015


rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.


================
Comment at: ELF/Writer.cpp:764
@@ +763,3 @@
+        Out = Section->getOutputSection();
+        assert(Section != nullptr && Out != nullptr);
+        ESym->st_shndx = Out->getSectionIndex();
----------------
You have just used Section :-)


http://reviews.llvm.org/D12944





More information about the llvm-commits mailing list