[PATCH] D128382: [LLD] Two tweaks to symbol ordering scheme

[Fangrui Song via Phabricator via llvm-commits]

MaskRay requested changes to this revision.
MaskRay added a comment.
This revision now requires changes to proceed.

How does it save one page?

4 size=0xC00000 input sections are too large (48MiB). If just for `ARM::getThunkSectionSpacing` (16MiB), it is too much.

Please check split-file. I think we need tests when ` totalSize >= target->getThunkSectionSpacing()` triggers or not.

Please answer why `executableInputSections > 1` is needed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128382