[llvm] r216119 - X86: Align the stack on word boundaries in LowerFormalArguments()

[Thomas Jablin]

I can confirm that r214980 is sufficient. I also should have checked the
testcase still failed. Sorry.


On Thu, Aug 21, 2014 at 6:51 PM, Duncan P. N. Exon Smith <
dexonsmith at apple.com> wrote:

> +tjablin
>
> > On 2014-Aug-21, at 15:42, Reid Kleckner <rnk at google.com> wrote:
> >
> > How was this not handled by r214980?
>
> It was.  Reverted in r216246.
>
> I looked carefully at this on August 4th (before r214980), and when it
> popped back up I didn't check that the testcase still failed.
>
> Thomas: I left your testcase in tree (and it still passes).  Can you
> confirm that r214980 is sufficient?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140821/79d90e91/attachment.html>