[PATCH] D32783: [PartialInlining] Add frequency based cost analysis

[Easwaran Raman via Phabricator via llvm-commits]

eraman accepted this revision.
eraman added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Transforms/IPO/PartialInlining.cpp:458
+
+  BranchProbability RelativeFreq =
+      getOutliningCallBBRelativeFreq(F, OI, Callee, CalleeBFI, OutliningCallBB);
----------------
eraman wrote:
> auto here and in the next line.
I think you can use auto  for NormWeightedRCost also.


https://reviews.llvm.org/D32783