Transforms: reapply SVN r219899
authorSaleem Abdulrasool <compnerd@compnerd.org>
Tue, 28 Oct 2014 18:27:37 +0000 (18:27 +0000)
committerSaleem Abdulrasool <compnerd@compnerd.org>
Tue, 28 Oct 2014 18:27:37 +0000 (18:27 +0000)
commitd178ada55ea4cc5d71443ec27495092fbb72b230
treeec71ccd72ffdb4eac61697c6c45fd9057d6e5e95
parent4f2fe82b6de2069ae48bd1e8df0e29826ad63351
Transforms: reapply SVN r219899

This restores the commit from SVN r219899 with an additional change to ensure
that the CodeGen is correct for the case that was identified as being incorrect
(originally PR7272).

In the case that during inlining we need to synthesize a value on the stack
(i.e. for passing a value byval), then any function involving that alloca must
be stripped of its tailness as the restriction that it does not access the
parent's stack no longer holds.  Unfortunately, a single alloca can cause a
rippling effect through out the inlining as the value may be aliased or may be
mutated through an escaped external call.  As such, we simply track if an alloca
has been introduced in the frame during inlining, and strip any tail calls.

llvm-svn: 220811
llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
llvm/lib/Transforms/Utils/InlineFunction.cpp
llvm/test/Transforms/Inline/byval-tail-call.ll
llvm/test/Transforms/Inline/inline-tail.ll
llvm/test/Transforms/Inline/inlined-allocas.ll [new file with mode: 0644]
llvm/test/Transforms/TailCallElim/basic.ll
llvm/test/Transforms/TailCallElim/byval.ll [new file with mode: 0644]