[PATCH] D70655: [AutoFDO] Top-down Inlining for specialization with context-sensitive profile

[Yuanfang Chen via Phabricator via llvm-commits]

ychen added inline comments.
Herald added subscribers: hoy, modimo, lxfind.


================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:1826
       &getAnalysis<ProfileSummaryInfoWrapperPass>().getPSI();
-  return SampleLoader.runOnModule(M, nullptr, PSI);
+  return SampleLoader.runOnModule(M, nullptr, PSI, nullptr);
 }
----------------
The new pass manager computes the call graph but here it is skipped. Should we also compute the call graph here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70655