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

[YongKang Zhu via Phabricator via llvm-commits]

yozhu added a comment.

In D128382#3603513 <https://reviews.llvm.org/D128382#3603513>, @MaskRay wrote:

> 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.

The original test covers the scenario where `totalSize < target->getThunkSectionSpacing()`.  The new test covers the opposite.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128382