aboutsummaryrefslogtreecommitdiff
path: root/src/cmd/vet
diff options
context:
space:
mode:
authorRuss Cox <rsc@golang.org>2019-05-08 13:16:05 -0400
committerRuss Cox <rsc@golang.org>2019-05-09 21:13:19 +0000
commitfe67ce2efb243b9289e81a61a5ccae79b75048f2 (patch)
tree88d9ceb8abb44da267f99e7e78b5532473002557 /src/cmd/vet
parent6ed2ec4aa5ea6c34b1e3d42b7995c81ab74a27c4 (diff)
downloadgo-fe67ce2efb243b9289e81a61a5ccae79b75048f2.tar.gz
go-fe67ce2efb243b9289e81a61a5ccae79b75048f2.zip
runtime: fix vet complaints for linux/amd64
Working toward making the tree vet-safe instead of having so many exceptions in cmd/vet/all/whitelist. This CL makes "GOOS=linux GOARCH=amd64 go vet -unsafeptr=false runtime" happy, while keeping "GO_BUILDER_NAME=misc-vetall go tool dist test" happy too. For #31916. Change-Id: I4ca1acb02f4666b102d25fcc55fac96b8f80379a Reviewed-on: https://go-review.googlesource.com/c/go/+/176100 Run-TryBot: Russ Cox <rsc@golang.org> Reviewed-by: Austin Clements <austin@google.com>
Diffstat (limited to 'src/cmd/vet')
-rw-r--r--src/cmd/vet/all/whitelist/amd64.txt15
-rw-r--r--src/cmd/vet/all/whitelist/android_amd64.txt3
-rw-r--r--src/cmd/vet/all/whitelist/darwin_amd64.txt3
-rw-r--r--src/cmd/vet/all/whitelist/dragonfly_amd64.txt1
-rw-r--r--src/cmd/vet/all/whitelist/freebsd_amd64.txt1
-rw-r--r--src/cmd/vet/all/whitelist/illumos_amd64.txt1
-rw-r--r--src/cmd/vet/all/whitelist/linux_amd64.txt3
-rw-r--r--src/cmd/vet/all/whitelist/nacl_amd64p32.txt1
-rw-r--r--src/cmd/vet/all/whitelist/netbsd_386.txt3
-rw-r--r--src/cmd/vet/all/whitelist/netbsd_amd64.txt5
-rw-r--r--src/cmd/vet/all/whitelist/openbsd_386.txt1
-rw-r--r--src/cmd/vet/all/whitelist/openbsd_amd64.txt3
-rw-r--r--src/cmd/vet/all/whitelist/plan9_amd64.txt3
-rw-r--r--src/cmd/vet/all/whitelist/solaris_amd64.txt1
-rw-r--r--src/cmd/vet/all/whitelist/windows_amd64.txt1
15 files changed, 0 insertions, 45 deletions
diff --git a/src/cmd/vet/all/whitelist/amd64.txt b/src/cmd/vet/all/whitelist/amd64.txt
deleted file mode 100644
index 5ce30cc6c2..0000000000
--- a/src/cmd/vet/all/whitelist/amd64.txt
+++ /dev/null
@@ -1,15 +0,0 @@
-// amd64-specific vet whitelist. See readme.txt for details.
-
-// False positives.
-
-// reflect trampolines intentionally omit arg size. Same for morestack.
-runtime/asm_amd64.s: [amd64] morestack: use of 8(SP) points beyond argument frame
-runtime/asm_amd64.s: [amd64] morestack: use of 16(SP) points beyond argument frame
-runtime/asm_amd64.s: [amd64] morestack: use of 8(SP) points beyond argument frame
-
-// Intentionally missing declarations. These are special assembly routines.
-// Some are jumped into from other routines, with values in specific registers.
-// duff* have direct calls from the compiler.
-// Others use the platform ABI.
-// There is no sensible corresponding Go prototype.
-runtime/asm_amd64.s: [amd64] aeshashbody: function aeshashbody missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/android_amd64.txt b/src/cmd/vet/all/whitelist/android_amd64.txt
deleted file mode 100644
index 90dabb0209..0000000000
--- a/src/cmd/vet/all/whitelist/android_amd64.txt
+++ /dev/null
@@ -1,3 +0,0 @@
-// android/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_linux_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/darwin_amd64.txt b/src/cmd/vet/all/whitelist/darwin_amd64.txt
deleted file mode 100644
index fcdacb2dc1..0000000000
--- a/src/cmd/vet/all/whitelist/darwin_amd64.txt
+++ /dev/null
@@ -1,3 +0,0 @@
-// darwin/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_darwin_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/dragonfly_amd64.txt b/src/cmd/vet/all/whitelist/dragonfly_amd64.txt
index 6c44159793..0db5503ef0 100644
--- a/src/cmd/vet/all/whitelist/dragonfly_amd64.txt
+++ b/src/cmd/vet/all/whitelist/dragonfly_amd64.txt
@@ -1,6 +1,5 @@
// dragonfly/amd64-specific vet whitelist. See readme.txt for details.
-runtime/sys_dragonfly_amd64.s: [amd64] settls: function settls missing Go declaration
syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 8(SP) should be num+0(FP)
syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 16(SP) should be a1+8(FP)
diff --git a/src/cmd/vet/all/whitelist/freebsd_amd64.txt b/src/cmd/vet/all/whitelist/freebsd_amd64.txt
index a910f48ca5..356be022cc 100644
--- a/src/cmd/vet/all/whitelist/freebsd_amd64.txt
+++ b/src/cmd/vet/all/whitelist/freebsd_amd64.txt
@@ -1,6 +1,5 @@
// freebsd/amd64-specific vet whitelist. See readme.txt for details.
-runtime/sys_freebsd_amd64.s: [amd64] settls: function settls missing Go declaration
syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 8(SP) should be num+0(FP)
syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 16(SP) should be a1+8(FP)
syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 24(SP) should be a2+16(FP)
diff --git a/src/cmd/vet/all/whitelist/illumos_amd64.txt b/src/cmd/vet/all/whitelist/illumos_amd64.txt
index 4525c03cd7..30f3912e3e 100644
--- a/src/cmd/vet/all/whitelist/illumos_amd64.txt
+++ b/src/cmd/vet/all/whitelist/illumos_amd64.txt
@@ -1,6 +1,5 @@
// illumos/amd64-specific vet whitelist. See readme.txt for details.
-runtime/sys_solaris_amd64.s: [amd64] settls: function settls missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] pipe1: function pipe1 missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] asmsysvicall6: function asmsysvicall6 missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] usleep2: function usleep2 missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/linux_amd64.txt b/src/cmd/vet/all/whitelist/linux_amd64.txt
deleted file mode 100644
index 2f4a933553..0000000000
--- a/src/cmd/vet/all/whitelist/linux_amd64.txt
+++ /dev/null
@@ -1,3 +0,0 @@
-// linux/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_linux_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/nacl_amd64p32.txt b/src/cmd/vet/all/whitelist/nacl_amd64p32.txt
index b3d6aa40a1..fb95a360a6 100644
--- a/src/cmd/vet/all/whitelist/nacl_amd64p32.txt
+++ b/src/cmd/vet/all/whitelist/nacl_amd64p32.txt
@@ -10,7 +10,6 @@ runtime/sys_nacl_amd64p32.s: [amd64p32] sigtramp: unknown variable ctxt
runtime/sys_nacl_amd64p32.s: [amd64p32] sigtramp: unknown variable ctxt
runtime/sys_nacl_amd64p32.s: [amd64p32] nacl_sysinfo: function nacl_sysinfo missing Go declaration
runtime/sys_nacl_amd64p32.s: [amd64p32] nacl_clock_gettime: function nacl_clock_gettime missing Go declaration
-runtime/sys_nacl_amd64p32.s: [amd64p32] settls: function settls missing Go declaration
// Clearer using FP than SP, but that requires named offsets.
runtime/asm_amd64p32.s: [amd64p32] rt0_go: unknown variable argc
diff --git a/src/cmd/vet/all/whitelist/netbsd_386.txt b/src/cmd/vet/all/whitelist/netbsd_386.txt
index a85c6e0f78..608021cae4 100644
--- a/src/cmd/vet/all/whitelist/netbsd_386.txt
+++ b/src/cmd/vet/all/whitelist/netbsd_386.txt
@@ -1,8 +1,5 @@
// netbsd/386-specific vet whitelist. See readme.txt for details.
-runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] settls: function settls missing Go declaration
-runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] sigreturn_tramp: function sigreturn_tramp missing Go declaration
-
runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 140(SP) points beyond argument frame
runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 4(SP) points beyond argument frame
runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 4(SP) points beyond argument frame
diff --git a/src/cmd/vet/all/whitelist/netbsd_amd64.txt b/src/cmd/vet/all/whitelist/netbsd_amd64.txt
deleted file mode 100644
index 23b6a8c0ad..0000000000
--- a/src/cmd/vet/all/whitelist/netbsd_amd64.txt
+++ /dev/null
@@ -1,5 +0,0 @@
-// netbsd/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] sigreturn_tramp: function sigreturn_tramp missing Go declaration
-
-runtime/sys_netbsd_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/openbsd_386.txt b/src/cmd/vet/all/whitelist/openbsd_386.txt
index 66172ff008..bfbf4d9c56 100644
--- a/src/cmd/vet/all/whitelist/openbsd_386.txt
+++ b/src/cmd/vet/all/whitelist/openbsd_386.txt
@@ -3,7 +3,6 @@
runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable signo
runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable info
runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable context
-runtime/sys_openbsd_386.s: [386] settls: function settls missing Go declaration
syscall/asm_unix_386.s: [386] Syscall: 8(SP) should be a1+4(FP)
syscall/asm_unix_386.s: [386] Syscall: 4(SP) should be trap+0(FP)
syscall/asm_unix_386.s: [386] Syscall6: 8(SP) should be a1+4(FP)
diff --git a/src/cmd/vet/all/whitelist/openbsd_amd64.txt b/src/cmd/vet/all/whitelist/openbsd_amd64.txt
deleted file mode 100644
index 433f62ca07..0000000000
--- a/src/cmd/vet/all/whitelist/openbsd_amd64.txt
+++ /dev/null
@@ -1,3 +0,0 @@
-// openbsd/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_openbsd_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/plan9_amd64.txt b/src/cmd/vet/all/whitelist/plan9_amd64.txt
deleted file mode 100644
index 1390475c77..0000000000
--- a/src/cmd/vet/all/whitelist/plan9_amd64.txt
+++ /dev/null
@@ -1,3 +0,0 @@
-// plan9/amd64-specific vet whitelist. See readme.txt for details.
-
-runtime/sys_plan9_amd64.s: [amd64] settls: function settls missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/solaris_amd64.txt b/src/cmd/vet/all/whitelist/solaris_amd64.txt
index 26a9da4271..19f66f7588 100644
--- a/src/cmd/vet/all/whitelist/solaris_amd64.txt
+++ b/src/cmd/vet/all/whitelist/solaris_amd64.txt
@@ -1,6 +1,5 @@
// solaris/amd64-specific vet whitelist. See readme.txt for details.
-runtime/sys_solaris_amd64.s: [amd64] settls: function settls missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] pipe1: function pipe1 missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] asmsysvicall6: function asmsysvicall6 missing Go declaration
runtime/sys_solaris_amd64.s: [amd64] usleep2: function usleep2 missing Go declaration
diff --git a/src/cmd/vet/all/whitelist/windows_amd64.txt b/src/cmd/vet/all/whitelist/windows_amd64.txt
index 4e6da405e3..348afd152e 100644
--- a/src/cmd/vet/all/whitelist/windows_amd64.txt
+++ b/src/cmd/vet/all/whitelist/windows_amd64.txt
@@ -3,4 +3,3 @@
runtime/sys_windows_amd64.s: [amd64] ctrlhandler: RET without writing to 4-byte ret+8(FP)
runtime/sys_windows_amd64.s: [amd64] callbackasm1: function callbackasm1 missing Go declaration
runtime/sys_windows_amd64.s: [amd64] tstart_stdcall: RET without writing to 4-byte ret+8(FP)
-runtime/sys_windows_amd64.s: [amd64] settls: function settls missing Go declaration