[PATCH] D57089: Provide reason messages for unviable inlining

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 28 00:51:30 PST 2019


yrouban updated this revision to Diff 183806.
yrouban added a comment.

Addressed comments:

- added a simple test
- changed implicit conversion to char* to explicit field access //IsViable.message//


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

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
  test/Transforms/Inline/inline-remark.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57089.183806.patch
Type: text/x-patch
Size: 5920 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190128/9fa92b6a/attachment.bin>


More information about the llvm-commits mailing list