[PATCH] D79590: [lld] Add a new output section ".text.unknown" for funtions with unknown hotness info especially in sampleFDO
    Wei Mi via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu May  7 12:28:33 PDT 2020
    
    
  
wmi marked 2 inline comments as done.
wmi added a comment.
Thanks for looking at it.
================
Comment at: lld/ELF/Writer.cpp:131
+  // ".text.unknown" means the hotness of the section is unknown. When
+  // SampleFDO is used, if function doesn't have sample, it could be very
+  // cold or it could be new function never being sampled. Those functions
----------------
MaskRay wrote:
> a function
Fixed.
================
Comment at: lld/ELF/Writer.cpp:132
+  // SampleFDO is used, if function doesn't have sample, it could be very
+  // cold or it could be new function never being sampled. Those functions
+  // will be kept in the ".text.unknown" section.
----------------
MaskRay wrote:
> a new function
Fixed.
Repository:
  rLLD LLVM Linker
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79590/new/
https://reviews.llvm.org/D79590
    
    
More information about the llvm-commits
mailing list