[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