[PATCH] D18560: [TTI] Add getInliningThresholdMultiplier.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 14 18:42:01 PDT 2016


hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

In http://reviews.llvm.org/D18560#401680, @jlebar wrote:

> Hal, I hope I didn't misunderstand: I took your comment to suggest that the multiplier should be fixed for a particular target -- it shouldn't depend on the caller/callee.  I can add that back if that's not what you had in mind.


This is fine. We can add those things back if we come up with a use case. As discussed, we really need to rework a bunch of this logic. LGTM.


http://reviews.llvm.org/D18560





More information about the llvm-commits mailing list