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

[Andrew Ng via Phabricator via llvm-commits]

andrewng added a comment.

The default was SHT_PROGBITS in the past, and was changed to SHT_NOBITS to "reduce file size". However, it's not clear that it actually reduces output file size, but only reduces the value of file size in the output. In any case, it does cause the alignment problem described and shown in the symbol-only-align.test test.

I intend to fix the orphan behaviour in a separate change, which is why it has been marked as XFAIL for now.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60131/new/

https://reviews.llvm.org/D60131