[PATCH] D87840: [lld] Add a new known text prefix - ".text.split."

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 10:52:27 PDT 2020


MaskRay added a comment.

This looks good to me. Please add a test to test/ELF/text-section-prefix.s



================
Comment at: lld/ELF/Writer.cpp:138
+  // profile inaccuracy. Techniques such as hugepage remapping can make
+  // conservative decisions at the section granularity. Additionally we find
+  // small improvement in icache and tlb metrics due to improved locality.
----------------
` Additionally we find small improvement in icache and tlb metrics due to improved locality.` might be too specific. It can probably be removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87840



More information about the llvm-commits mailing list