[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