Commit 7d80a2ea authored by Elias Naur's avatar Elias Naur

runtime: allow crash() to raise SIGABRT on darwin/arm64

To avoid gigantic core dumps, the runtime avoids raising SIGABRT
on crashes on 64-bit Darwin systems. Mobile OS'es (probably) don't
generate huge core dumps, so to aid crash reporters, allow SIGABRT
on crashes on darwin/arm64.

Change-Id: I4a29608f400967d76f9bd0643fea22244c2da9df
Reviewed-on: https://go-review.googlesource.com/49770
Run-TryBot: Elias Naur <elias.naur@gmail.com>
Reviewed-by: default avatarAvelino <t@avelino.xxx>
Reviewed-by: default avatarIan Lance Taylor <iant@golang.org>
parent c9f8a6cd
...@@ -8,7 +8,6 @@ package runtime ...@@ -8,7 +8,6 @@ package runtime
import ( import (
"runtime/internal/atomic" "runtime/internal/atomic"
"runtime/internal/sys"
"unsafe" "unsafe"
) )
...@@ -474,7 +473,7 @@ func crash() { ...@@ -474,7 +473,7 @@ func crash() {
// this means the OS X core file will be >128 GB and even on a zippy // this means the OS X core file will be >128 GB and even on a zippy
// workstation can take OS X well over an hour to write (uninterruptible). // workstation can take OS X well over an hour to write (uninterruptible).
// Save users from making that mistake. // Save users from making that mistake.
if sys.PtrSize == 8 { if GOARCH == "amd64" {
return return
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment