[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