[PATCH] D87132: [Partial Inliner] Compute intrinsic cost through TTI

Dangeti Tharun kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 4 02:21:59 PDT 2020


tdangeti created this revision.
tdangeti added reviewers: mtrofin, venur, sekharbvrs.
tdangeti added projects: LLVM, clang.
Herald added subscribers: llvm-commits, hiraditya.
tdangeti requested review of this revision.

https://bugs.llvm.org/show_bug.cgi?id=45932

assert(OutlinedFunctionCost >= Cloner.OutlinedRegionCost && "Outlined function cost should be no less than the outlined region") getting triggered in computeBBInlineCost.

Intrinsics like "assume" are considered regular function calls while computing costs.
This patch enables computeBBInlineCost to queries TTI for intrinsic call cost.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D87132

Files:
  llvm/lib/Transforms/IPO/PartialInlining.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87132.289898.patch
Type: text/x-patch
Size: 8045 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200904/3fde67d7/attachment.bin>


More information about the llvm-commits mailing list