[PATCH] D36054: Emit only A Single Opt Remark When Inlining

[Sam Elliott via Phabricator via llvm-commits]

lenary updated this revision to Diff 111758.
lenary added a comment.

Address review feedback.

Using Optional<InlineCost> seemed to be a good compromise,
so I have used it.

I also am now emitting more optimisation remarks in the new
pass.


https://reviews.llvm.org/D36054

Files:
  include/llvm/Analysis/InlineCost.h
  lib/Transforms/IPO/Inliner.cpp
  test/LTO/Resolution/X86/diagnostic-handler-remarks-with-hotness.ll
  test/LTO/Resolution/X86/diagnostic-handler-remarks.ll
  test/LTO/X86/diagnostic-handler-remarks-with-hotness.ll
  test/LTO/X86/diagnostic-handler-remarks.ll
  test/ThinLTO/X86/diagnostic-handler-remarks-with-hotness.ll
  test/ThinLTO/X86/diagnostic-handler-remarks.ll
  test/Transforms/Inline/optimization-remarks-passed-yaml.ll
  test/Transforms/Inline/optimization-remarks-with-hotness.ll
  test/Transforms/Inline/optimization-remarks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36054.111758.patch
Type: text/x-patch
Size: 16221 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170818/3b3d06bf/attachment.bin>