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

YongKang Zhu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 15:34:40 PDT 2022


yozhu added a comment.

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

> I agree that the mentioned cases do not unnecessarily need reordering. It doesn't matter either way, then why change the behavior with more code?

This could potentially save one page that hot code occupies when branch thunk is definitely not required.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128382



More information about the llvm-commits mailing list