[PATCH] D60131: [ELF] Change default output section type to SHT_PROGBITS

[Andrew Ng via Phabricator via llvm-commits]

andrewng created this revision.
andrewng added reviewers: ruiu, grimar, jakehehrlich.
Herald added subscribers: MaskRay, arichardson, javed.absar, emaste.
Herald added a reviewer: espindola.

This fixes an issue where a symbol only section at the start of a
PT_LOAD segment, causes incorrect alignment of the file offset for the
start of the segment. This results in the output of an invalid ELF.

This change breaks an existing test related to orphan sections. See test
orphan-phdrs.s for details.


https://reviews.llvm.org/D60131

Files:
  ELF/LinkerScript.cpp
  test/ELF/linkerscript/arm-exidx-order.test
  test/ELF/linkerscript/extend-pt-load2.test
  test/ELF/linkerscript/merge-sections.s
  test/ELF/linkerscript/orphan-phdrs.s
  test/ELF/linkerscript/symbol-only-align.test
  test/ELF/linkerscript/symbol-only-flags.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60131.193294.patch
Type: text/x-patch
Size: 4927 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190402/941c66c1/attachment.bin>