[LLVMbugs] [Bug 2921] New: alloca >4k stack probes on windows/visual studio

[bugzilla-daemon at cs.uiuc.edu]

http://llvm.org/bugs/show_bug.cgi?id=2921

           Summary: alloca >4k stack probes on windows/visual studio
           Product: libraries
           Version: 2.3
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Backend: X86
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: ubub at gmx.net
                CC: llvmbugs at cs.uiuc.edu


llvm already contains code to insert the required >4k stack probes aka
alloca_probe/chkstk on cygwin/mingw builds.
but if llvm is build with visual studio the insertion is omitted.

I've changed the test in lib/Target/X86/X86RegisterInfo.cpp:

if (NumBytes >= 4096 && Subtarget->isTargetCygMing()) {

to:

if (NumBytes >= 4096 && (Subtarget->isTargetCygMing() ||
Subtarget->isTargetWindows())) {

to avoid crashes for stack allocation bigger 4k.


-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.