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

[Fangrui Song via Phabricator via llvm-commits]

MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

> The linker today will always put hot contributions in the middle of cold ones when targeting RISC machine, so to minimize the chances that branch thunks need be generated for hot code calling into cold code.

Mention --symbol-ordering-file in the subject or the commit message, otherwise readers may thought this applies to normal usage.



================
Comment at: lld/ELF/Writer.cpp:1363
+  //
+  // The above is not necessary if total size of output section is small. Note
+  // that we assume all input sections are executable if the output section is
----------------
total size of isd




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128382