[PATCH] D62540: [SampleFDO] For functions without profiles, provide an option to put them in a special text section
    Wei Mi via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue May 28 14:59:07 PDT 2019
    
    
  
wmi added a comment.
> Longer term, the plan is to put a white list of the symbols into the profile data so that the compiler can decide if a function is newly created or simply cold.
> 
> Wei, if that is in place, is there a need for this patch?
Yes, that is the longer term plan, and that should bring extra benefit that compiler optimizations will be benefited from the extra information.
We use the current patch as a temporary solution because we saw some urgent demand to save a lot of memory from it. It won't be needed if the longer term plan is in place.
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62540/new/
https://reviews.llvm.org/D62540
    
    
More information about the llvm-commits
mailing list