[PATCH] D81016: Adding InlineCostAnnotationPrinterPass for Inline Cost Analysis

[Kirill Naumov via Phabricator via llvm-commits]

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

Addressed @mtrofin 's comments:

  - Before the change, we had CostAnnotationWriter that was used just to display the cost difference per instruction, hence the CostThresholdMap (which now became InstructionCostDetailMap) was in the printer itself. Now, the change changed that by putting the map into InlineCostCallAnalyzer to keep all the data in this class.
- The move of code was unnecessary, changed it


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

https://reviews.llvm.org/D81016

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: D81016.268243.patch
Type: text/x-patch
Size: 14014 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200603/3aff348c/attachment.bin>