[PATCH] D105349: [llvm][Inline] Add interface to return cost-benefit stuff

[Mircea Trofin via Phabricator via llvm-commits]

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

lgtm, one comment



================
Comment at: llvm/include/llvm/Analysis/InlineCost.h:64
+
+  APInt getCost() const { return Cost; }
+
----------------
you can return const APInt &, I think APInt is a bit larger than a pointer


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105349