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: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]
                CC: [email protected]


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.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to