[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