[PATCH] D19416: [ELF] - Align sections file offsets correctly.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 09:31:34 PDT 2016


ruiu added inline comments.

================
Comment at: ELF/Writer.cpp:1667-1670
@@ +1666,6 @@
+
+  // Relocatable output does not have program headers
+  // and does not need any other offset adjusting.
+  if (Config->Relocatable || !(Sec->getFlags() & SHF_ALLOC))
+    return Off;
+
----------------
For relocatable output, do we even have to adjust file offset?


http://reviews.llvm.org/D19416





More information about the llvm-commits mailing list