[PATCH] D49412: Enrich inline messages

[Yevgeny Rouban via Phabricator via llvm-commits]

yrouban marked an inline comment as not done.
yrouban added inline comments.


================
Comment at: include/llvm/Transforms/Utils/Cloning.h:218
+/// describes a reason why it is decided to not inline.
+struct InlineResult {
+  const char* message = nullptr;
----------------
tejohnson wrote:
> I think this would make more sense in InlineCost.h
InlineCost.h relates to //Analysis// but InlineResult is a result of a //transformations//.
In other words all users of InlineResult reside in lib/Transforms.
But I do not insist.


Repository:
  rL LLVM

https://reviews.llvm.org/D49412