[PATCH] D57089: Provide reason messages for unviable inlining

[Yevgeny Rouban via Phabricator via llvm-commits]

yrouban created this revision.
yrouban added reviewers: xbolva00, tejohnson, craig.topper, chandlerc, apilipenko, skatkov.
Herald added subscribers: haicheng, eraman, nhaehnle, jvesely, arsenm.

InlineCost's //isInlineViable()// is changed to return //InlineResult// instead of bool. This provides messages for failure reasons.
This allows to get more specific messages for cases where callsites are not viable for inlining.


https://reviews.llvm.org/D57089

Files:
  include/llvm/Analysis/InlineCost.h
  lib/Analysis/InlineCost.cpp
  lib/Target/AMDGPU/AMDGPUInline.cpp
  lib/Transforms/IPO/AlwaysInliner.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57089.183045.patch
Type: text/x-patch
Size: 5045 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190123/d584909a/attachment.bin>