[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]

wmi marked 3 inline comments as done.
wmi added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ProfileSummaryInfo.h:123
+  /// unknown.
+  bool isFunctionUnknownInCallGraph(const Function *F);
   /// Returns true if \p F contains hot code with regard to a given hot
----------------
davidxl wrote:
> --> isFunctionHotnessUnknown (or isFunctionProfileUnknown)  seems better. 
Ok, changed.


================
Comment at: llvm/lib/Analysis/ProfileSummaryInfo.cpp:196
+bool ProfileSummaryInfo::isFunctionUnknownInCallGraph(const Function *F) {
+  if (!F || !hasSampleProfile())
+    return false;
----------------
davidxl wrote:
> assert (F && hasSampleProfile()) ?
Added.


================
Comment at: llvm/lib/CodeGen/CodeGenPrepare.cpp:456
   if (ProfileGuidedSectionPrefix) {
+    bool IsUnknown = false;
     if (PSI->isFunctionHotInCallGraph(&F, *BFI))
----------------
davidxl wrote:
> is this used?
Removed.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D62540