[PATCH] D42846: [PartialInlining] Use isInlineViable to detect constructs preventing inlining.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

isInlineViable doesn't care whether the code is actually reachable, so you could pessimize partial inlining in certain cases, e.g. a function which calls va_start conditionally.  I don't think we need fix that now, but it would be nice to at least have a testcase demonstrating the issue.


https://reviews.llvm.org/D42846