[PATCH] D81743: InlineCostAnnotationPrinterPass - Introducing the pass

Kirill Naumov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 12:35:19 PDT 2020


knaumov updated this revision to Diff 270474.
knaumov added a comment.

Changed the creation of InlineParams back to default params and added a "to-do" comment.
For the purposes of this pass, the idea of which is to give access for non-debug builds to tests that use debug functionality, the InlineParams can be taken as default. My suggestion is to leave the extension of the pass to parametrization with different InlineParams as a "to-do" for now.


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

https://reviews.llvm.org/D81743

Files:
  llvm/include/llvm/Analysis/InlineCost.h
  llvm/lib/Analysis/InlineCost.cpp
  llvm/lib/Passes/PassRegistry.def
  llvm/test/Transforms/Inline/debuginline-cost-delta.ll
  llvm/test/Transforms/Inline/inline-cost-annotation-pass.ll
  llvm/test/Transforms/Inline/print-instructions-deltas-unfinished.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81743.270474.patch
Type: text/x-patch
Size: 8037 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200612/b743286e/attachment-0001.bin>


More information about the llvm-commits mailing list