[PATCH] chkstk for Windows

[Yaron Keren]

Hi,

I am getting build failures for x86_64_debian, x86_64_ubuntu, s390-linux1,
ppc2-linux2.


http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-debian-fast/builds/8571

http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-ubuntu-13.04/builds/8268
 http://lab.llvm.org:8011/builders/llvm-ppc64-linux2/builds/7807

The failing tests are:


   1. LLVM::mingw-alloca.ll<http://lab.llvm.org:8011/builders/llvm-ppc64-linux2/builds/7807/steps/test-llvm/logs/LLVM%3A%3Amingw-alloca.ll>
   2. LLVM::win64_alloca_dynalloca.ll<http://lab.llvm.org:8011/builders/llvm-ppc64-linux2/builds/7807/steps/test-llvm/logs/LLVM%3A%3Awin64_alloca_dynalloca.ll>

I think that the tests should be updated to reflect the change, right?

Yaron



2013/10/23 Anton Korobeynikov <anton at korobeynikov.info>

> LGTM
>
> On Wed, Oct 23, 2013 at 10:51 PM, Yaron Keren <yaron.keren at gmail.com>
> wrote:
> > Calling _chkstk is required on ELF as well as COFF on Windows. Without
> > _chkstk functions requiring large stack crash in initialization code.
> > Previous code tested for COFF format but not Mach-O and this patch
> modifies
> > the code to test for Windows OS.
> >
> > Credits to Andrew MacPherson.
> >
> > Yaron
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
>
>
>
> --
> With best regards, Anton Korobeynikov
> Faculty of Mathematics and Mechanics, Saint Petersburg State University
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131023/c95b6867/attachment.html>