[PATCH] D73994: [InlineCost] Relax bonus restrictions on uninlinable functions

[George Burgess IV via Phabricator via llvm-commits]

george.burgess.iv added a comment.

> I have concerns on so much plumbing changes to make the small heuristic change.

Agreed that that's discouraging. Lemme take numbers and see what this does for us. If this still looks promising, I'll split as requested.

> sharing performance numbers on standard benchmarks with O2 <https://reviews.llvm.org/owners/package/2/> & O2 <https://reviews.llvm.org/owners/package/2/> (thin) LTO,

Thanks for the feedback! I'm not sure what benchmarks are considered standard. Do we have documentation on this, or might you have specific ones in mind I can try this on?


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

https://reviews.llvm.org/D73994