[PATCH] D153154: [NFC][Inliner] Introduce another multiplier for cost benefit analysis and make multipliers overriddable in TargetTransformInfo.

Mingming Liu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 21 11:25:57 PDT 2023


mingmingl updated this revision to Diff 557191.
mingmingl marked an inline comment as done.
mingmingl retitled this revision from "[NFC][Inliner] Introduce per-target savings multiplier and profitable multiplier to use a hybrid of cost-benefit and cost-threshold analysis." to "[NFC][Inliner] Introduce another multiplier for cost benefit analysis and make multipliers overriddable in TargetTransformInfo.".
mingmingl edited the summary of this revision.
mingmingl added a comment.

Revive this NFC patch to expose tunable knobs for workloads/hardwares that could be more frontend bound.

Address feedback.


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

https://reviews.llvm.org/D153154

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/lib/Analysis/InlineCost.cpp
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/test/Transforms/Inline/inline-cost-benefit-multiplier-override.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153154.557191.patch
Type: text/x-patch
Size: 9919 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230921/dd389214/attachment.bin>


More information about the llvm-commits mailing list