]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: enable -d=checkptr even on windows
authorAlex Brainman <alex.brainman@gmail.com>
Sat, 4 Apr 2020 07:36:58 +0000 (18:36 +1100)
committerAlex Brainman <alex.brainman@gmail.com>
Sun, 5 Apr 2020 06:04:27 +0000 (06:04 +0000)
CL 201783 enable -d=checkptr when -race or -msan is specified
everywhere but windows.

But, now that all unsafe pointer conversions in the standard
library are fixed, enable -d=checkptr even on windows.

Updates #34964
Updates #34972

Change-Id: Id912fa83b0d5b46c6f1c134c742fd94d2d185835
Reviewed-on: https://go-review.googlesource.com/c/go/+/227003
Run-TryBot: Alex Brainman <alex.brainman@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/cmd/compile/internal/gc/main.go

index f65b95a9c9e49ddcee0f7caa3b6ecff616227d9e..2152c619fa82fa4352c40fbf06e5fb3b6ca3e486 100644 (file)
@@ -379,9 +379,8 @@ func Main(archInit func(*Arch)) {
        if flag_race && flag_msan {
                log.Fatal("cannot use both -race and -msan")
        }
-       if (flag_race || flag_msan) && objabi.GOOS != "windows" {
-               // -race and -msan imply -d=checkptr for now (except on windows).
-               // TODO(mdempsky): Re-evaluate before Go 1.14. See #34964.
+       if flag_race || flag_msan {
+               // -race and -msan imply -d=checkptr for now.
                Debug_checkptr = 1
        }
        if ispkgin(omit_pkgs) {