[PATCH] D49412: Enrich inline messages
Yevgeny Rouban via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 18 00:43:49 PDT 2018
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
More information about the llvm-commits
mailing list