[PATCH] D95199: [ELF] Write output sections in PT_LOAD segment order

[Patrick Oppenlander via Phabricator via llvm-commits]

pattop added a comment.

In D95199#2534394 <https://reviews.llvm.org/D95199#2534394>, @peter.smith wrote:

> Thanks for the update, I'll try and take a look later in the week. Will have to have a think about the zero size section case. If we haven't assigned it to a PT_LOAD and it defines no symbol it may be possible to not put it in the object file.

No problem. Hopefully my comments on the test cases make some sense. Unfortunately some of your code comments moved around a bit when I updated the patch.

Doing something with zero size sections could be a nice cleanup, but it looks like it would cause some churn in the tests. Either way, I think it's separate to this issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95199