replace __chkstk function with a stub that does not crash

Closes #508
See #302

2 files changed, 15 insertions, 36 deletions

diff --git a/std/special/bootstrap.zig b/std/special/bootstrap.zig
index 3067406431..fea062712b 100644
--- a/std/special/bootstrap.zig
+++ b/std/special/bootstrap.zig
@@ -5,12 +5,13 @@ const root = @import("@root");
 const std = @import("std");
 const builtin = @import("builtin");
 
+const is_windows = builtin.os == builtin.Os.windows;
 const want_main_symbol = builtin.link_libc;
-const want_start_symbol = !want_main_symbol;
+const want_start_symbol = !want_main_symbol and !is_windows;
+const want_WinMainCRTStartup = is_windows and !builtin.link_libc;
 
 var argc_ptr: &usize = undefined;
 
-const is_windows = builtin.os == builtin.Os.windows;
 
 export nakedcc fn _start() -> noreturn {
     if (!want_start_symbol) {
@@ -18,18 +19,6 @@ export nakedcc fn _start() -> noreturn {
         unreachable;
     }
 
-    if (is_windows) {
-        if (builtin.arch == builtin.Arch.x86_64) {
-            // Align the stack pointer to 16 bytes.
-            asm volatile (
-                \\ and    $0xfffffffffffffff0,%%rsp
-                \\ sub    $0x10,%%rsp
-                :::"rsp"
-            );
-        }
-        windowsCallMainAndExit()
-    }
-
     switch (builtin.arch) {
         builtin.Arch.x86_64 => {
             argc_ptr = asm("lea (%%rsp), %[argc]": [argc] "=r" (-> &usize));
@@ -42,8 +31,13 @@ export nakedcc fn _start() -> noreturn {
     posixCallMainAndExit()
 }
 
-fn windowsCallMainAndExit() -> noreturn {
+export fn WinMainCRTStartup() -> noreturn {
+    if (!want_WinMainCRTStartup) {
+        @setGlobalLinkage(WinMainCRTStartup, builtin.GlobalLinkage.Internal);
+        unreachable;
+    }
     @setAlignStack(16);
+
     std.debug.user_main_fn = root.main;
     root.main() %% std.os.windows.ExitProcess(1);
     std.os.windows.ExitProcess(0);
diff --git a/std/special/compiler_rt/index.zig b/std/special/compiler_rt/index.zig
index 36ad997120..f27e3069f0 100644
--- a/std/special/compiler_rt/index.zig
+++ b/std/special/compiler_rt/index.zig
@@ -155,32 +155,17 @@ export nakedcc fn _chkstk() align(4) {
     @setGlobalLinkage(_chkstk, builtin.GlobalLinkage.Internal);
 }
 
-export nakedcc fn __chkstk() align(4) {
+// TODO The implementation from compiler-rt causes crashes and
+// the implementation from disassembled ntdll seems to depend on
+// thread local storage. So we have given up this safety check
+// and simply have `ret`.
+export nakedcc fn __chkstk() align(8) {
     @setDebugSafety(this, false);
 
     if (win64_nocrt) {
         @setGlobalLinkage(__chkstk, strong_linkage);
         asm volatile (
-            \\         push   %%rcx
-            \\         cmp    $0x1000,%%rax
-            \\         lea    16(%%rsp),%%rcx     // rsp before calling this routine -> rcx
-            \\         jb     1f
-            \\ 2:
-            \\         sub    $0x1000,%%rcx
-            \\         test   %%rcx,(%%rcx)
-            \\         sub    $0x1000,%%rax
-            \\         cmp    $0x1000,%%rax
-            \\         ja     2b
-            \\ 1:
-            \\         sub    %%rax,%%rcx
-            \\         test   %%rcx,(%%rcx)
-            \\ 
-            \\         lea    8(%%rsp),%%rax     // load pointer to the return address into rax
-            \\         mov    %%rcx,%%rsp        // install the new top of stack pointer into rsp
-            \\         mov    -8(%%rax),%%rcx    // restore rcx
-            \\         push   (%%rax)           // push return address onto the stack
-            \\         sub    %%rsp,%%rax        // restore the original value in rax
-            \\         ret
+            \\ ret
         );
         unreachable;
     }