From 902da52f7b5826c2083effc20b209f39d58d2919 Mon Sep 17 00:00:00 2001 From: Than McIntosh Date: Fri, 30 Sep 2022 14:36:58 +0000 Subject: [PATCH] Revert "cmd/cgo, cmd/compile, cmd/link: remove old style build tags" This reverts commit 66165739828b8326b625463beb51ae510b2ab8dd, corresponding to CL 436915. Reason for revert: this is causing some bootstrap build problems with older versions of Go 1.17, as I understand it. Still under investigation. Change-Id: Idb6e17ff7b47004cbf87f967af6d84f214d8abb5 Reviewed-on: https://go-review.googlesource.com/c/go/+/435471 Reviewed-by: David Chase --- src/cmd/cgo/ast_go1.go | 1 + src/cmd/cgo/ast_go118.go | 1 + src/cmd/compile/internal/base/bootstrap_false.go | 1 + src/cmd/compile/internal/base/bootstrap_true.go | 1 + src/cmd/compile/internal/base/mapfile_mmap.go | 1 + src/cmd/compile/internal/base/mapfile_read.go | 7 +++++++ src/cmd/compile/internal/ir/mknode.go | 1 + src/cmd/compile/internal/ssa/flags_test.go | 1 + src/cmd/compile/internal/ssa/gen/386Ops.go | 1 + src/cmd/compile/internal/ssa/gen/AMD64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/ARM64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/ARMOps.go | 1 + src/cmd/compile/internal/ssa/gen/LOONG64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/MIPS64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/MIPSOps.go | 1 + src/cmd/compile/internal/ssa/gen/PPC64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/RISCV64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/S390XOps.go | 1 + src/cmd/compile/internal/ssa/gen/WasmOps.go | 1 + src/cmd/compile/internal/ssa/gen/dec64Ops.go | 1 + src/cmd/compile/internal/ssa/gen/decOps.go | 1 + src/cmd/compile/internal/ssa/gen/genericOps.go | 1 + src/cmd/compile/internal/ssa/gen/main.go | 1 + src/cmd/compile/internal/ssa/gen/rulegen.go | 1 + src/cmd/compile/internal/typecheck/builtin/runtime.go | 1 + src/cmd/compile/internal/typecheck/mkbuiltin.go | 1 + src/cmd/compile/internal/types2/example_test.go | 1 + src/cmd/internal/bio/buf_mmap.go | 1 + src/cmd/internal/bio/buf_nommap.go | 7 +++++++ src/cmd/link/elf_test.go | 1 + src/cmd/link/internal/ld/outbuf_mmap.go | 1 + src/cmd/link/internal/ld/outbuf_nofallocate.go | 1 + src/cmd/link/internal/ld/outbuf_nommap.go | 9 +++++++++ 33 files changed, 53 insertions(+) diff --git a/src/cmd/cgo/ast_go1.go b/src/cmd/cgo/ast_go1.go index ce61d29095..f52bf00d7c 100644 --- a/src/cmd/cgo/ast_go1.go +++ b/src/cmd/cgo/ast_go1.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build compiler_bootstrap +// +build compiler_bootstrap package main diff --git a/src/cmd/cgo/ast_go118.go b/src/cmd/cgo/ast_go118.go index 9f759b8ee5..db0108ed72 100644 --- a/src/cmd/cgo/ast_go118.go +++ b/src/cmd/cgo/ast_go118.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build !compiler_bootstrap +// +build !compiler_bootstrap package main diff --git a/src/cmd/compile/internal/base/bootstrap_false.go b/src/cmd/compile/internal/base/bootstrap_false.go index ea6da4348f..c77fcd7308 100644 --- a/src/cmd/compile/internal/base/bootstrap_false.go +++ b/src/cmd/compile/internal/base/bootstrap_false.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build !compiler_bootstrap +// +build !compiler_bootstrap package base diff --git a/src/cmd/compile/internal/base/bootstrap_true.go b/src/cmd/compile/internal/base/bootstrap_true.go index d0c6c88f56..1eb58b2f9d 100644 --- a/src/cmd/compile/internal/base/bootstrap_true.go +++ b/src/cmd/compile/internal/base/bootstrap_true.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build compiler_bootstrap +// +build compiler_bootstrap package base diff --git a/src/cmd/compile/internal/base/mapfile_mmap.go b/src/cmd/compile/internal/base/mapfile_mmap.go index 6c5a4213a2..b01d0f4efc 100644 --- a/src/cmd/compile/internal/base/mapfile_mmap.go +++ b/src/cmd/compile/internal/base/mapfile_mmap.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build darwin || dragonfly || freebsd || linux || netbsd || openbsd || (solaris && go1.20) +// +build darwin dragonfly freebsd linux netbsd openbsd solaris,go1.20 package base diff --git a/src/cmd/compile/internal/base/mapfile_read.go b/src/cmd/compile/internal/base/mapfile_read.go index 5ac6272dc1..e84810ea34 100644 --- a/src/cmd/compile/internal/base/mapfile_read.go +++ b/src/cmd/compile/internal/base/mapfile_read.go @@ -3,6 +3,13 @@ // license that can be found in the LICENSE file. //go:build !darwin && !dragonfly && !freebsd && !linux && !netbsd && !openbsd && !(solaris && go1.20) +// +build !darwin +// +build !dragonfly +// +build !freebsd +// +build !linux +// +build !netbsd +// +build !openbsd +// +build !solaris !go1.20 package base diff --git a/src/cmd/compile/internal/ir/mknode.go b/src/cmd/compile/internal/ir/mknode.go index caf4ba0135..68d8999f83 100644 --- a/src/cmd/compile/internal/ir/mknode.go +++ b/src/cmd/compile/internal/ir/mknode.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore // Note: this program must be run in this directory. // go run mknode.go diff --git a/src/cmd/compile/internal/ssa/flags_test.go b/src/cmd/compile/internal/ssa/flags_test.go index d0079ac5e8..0bc1097199 100644 --- a/src/cmd/compile/internal/ssa/flags_test.go +++ b/src/cmd/compile/internal/ssa/flags_test.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build amd64 || arm64 +// +build amd64 arm64 package ssa diff --git a/src/cmd/compile/internal/ssa/gen/386Ops.go b/src/cmd/compile/internal/ssa/gen/386Ops.go index 4b7fe9fe37..88e061151e 100644 --- a/src/cmd/compile/internal/ssa/gen/386Ops.go +++ b/src/cmd/compile/internal/ssa/gen/386Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/AMD64Ops.go b/src/cmd/compile/internal/ssa/gen/AMD64Ops.go index 65456a8f77..fc42fa5e28 100644 --- a/src/cmd/compile/internal/ssa/gen/AMD64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/AMD64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/ARM64Ops.go b/src/cmd/compile/internal/ssa/gen/ARM64Ops.go index 4ef949a179..0e6e6e4a5f 100644 --- a/src/cmd/compile/internal/ssa/gen/ARM64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/ARM64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/ARMOps.go b/src/cmd/compile/internal/ssa/gen/ARMOps.go index 99684e4e03..3803f273c1 100644 --- a/src/cmd/compile/internal/ssa/gen/ARMOps.go +++ b/src/cmd/compile/internal/ssa/gen/ARMOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/LOONG64Ops.go b/src/cmd/compile/internal/ssa/gen/LOONG64Ops.go index fdf42ec944..a59b592445 100644 --- a/src/cmd/compile/internal/ssa/gen/LOONG64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/LOONG64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/MIPS64Ops.go b/src/cmd/compile/internal/ssa/gen/MIPS64Ops.go index 59a9c8c714..7b18c42ffb 100644 --- a/src/cmd/compile/internal/ssa/gen/MIPS64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/MIPS64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/MIPSOps.go b/src/cmd/compile/internal/ssa/gen/MIPSOps.go index 17ae3d4251..523847badc 100644 --- a/src/cmd/compile/internal/ssa/gen/MIPSOps.go +++ b/src/cmd/compile/internal/ssa/gen/MIPSOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/PPC64Ops.go b/src/cmd/compile/internal/ssa/gen/PPC64Ops.go index 4428ea62a7..30a6f6fe07 100644 --- a/src/cmd/compile/internal/ssa/gen/PPC64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/PPC64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/RISCV64Ops.go b/src/cmd/compile/internal/ssa/gen/RISCV64Ops.go index 1d2d22643d..f09910782d 100644 --- a/src/cmd/compile/internal/ssa/gen/RISCV64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/RISCV64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/S390XOps.go b/src/cmd/compile/internal/ssa/gen/S390XOps.go index 29cb260d87..eef8a2557c 100644 --- a/src/cmd/compile/internal/ssa/gen/S390XOps.go +++ b/src/cmd/compile/internal/ssa/gen/S390XOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/WasmOps.go b/src/cmd/compile/internal/ssa/gen/WasmOps.go index 1ba94eaa5f..edfba4ee99 100644 --- a/src/cmd/compile/internal/ssa/gen/WasmOps.go +++ b/src/cmd/compile/internal/ssa/gen/WasmOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/dec64Ops.go b/src/cmd/compile/internal/ssa/gen/dec64Ops.go index 24b23c00b7..78fcea885a 100644 --- a/src/cmd/compile/internal/ssa/gen/dec64Ops.go +++ b/src/cmd/compile/internal/ssa/gen/dec64Ops.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/decOps.go b/src/cmd/compile/internal/ssa/gen/decOps.go index 5449727a8a..d5cd79378c 100644 --- a/src/cmd/compile/internal/ssa/gen/decOps.go +++ b/src/cmd/compile/internal/ssa/gen/decOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/genericOps.go b/src/cmd/compile/internal/ssa/gen/genericOps.go index df649eeece..d09b9aab75 100644 --- a/src/cmd/compile/internal/ssa/gen/genericOps.go +++ b/src/cmd/compile/internal/ssa/gen/genericOps.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore package main diff --git a/src/cmd/compile/internal/ssa/gen/main.go b/src/cmd/compile/internal/ssa/gen/main.go index fd00df680d..81740dcf1c 100644 --- a/src/cmd/compile/internal/ssa/gen/main.go +++ b/src/cmd/compile/internal/ssa/gen/main.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore // The gen command generates Go code (in the parent directory) for all // the architecture-specific opcodes, blocks, and rewrites. diff --git a/src/cmd/compile/internal/ssa/gen/rulegen.go b/src/cmd/compile/internal/ssa/gen/rulegen.go index d7b4cfa612..fc9b4c9b3e 100644 --- a/src/cmd/compile/internal/ssa/gen/rulegen.go +++ b/src/cmd/compile/internal/ssa/gen/rulegen.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build gen +// +build gen // This program generates Go code that applies rewrite rules to a Value. // The generated code implements a function of type func (v *Value) bool diff --git a/src/cmd/compile/internal/typecheck/builtin/runtime.go b/src/cmd/compile/internal/typecheck/builtin/runtime.go index 83695013c3..310148ce00 100644 --- a/src/cmd/compile/internal/typecheck/builtin/runtime.go +++ b/src/cmd/compile/internal/typecheck/builtin/runtime.go @@ -7,6 +7,7 @@ // to avoid depending on having a working compiler binary. //go:build ignore +// +build ignore package runtime diff --git a/src/cmd/compile/internal/typecheck/mkbuiltin.go b/src/cmd/compile/internal/typecheck/mkbuiltin.go index 4def95ebd6..aa3a94a19a 100644 --- a/src/cmd/compile/internal/typecheck/mkbuiltin.go +++ b/src/cmd/compile/internal/typecheck/mkbuiltin.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build ignore +// +build ignore // Generate builtin.go from builtin/runtime.go. diff --git a/src/cmd/compile/internal/types2/example_test.go b/src/cmd/compile/internal/types2/example_test.go index 9212c38560..ad0f22fcec 100644 --- a/src/cmd/compile/internal/types2/example_test.go +++ b/src/cmd/compile/internal/types2/example_test.go @@ -6,6 +6,7 @@ // access to compiled packages for import. // //go:build !arm && !arm64 +// +build !arm,!arm64 package types2_test diff --git a/src/cmd/internal/bio/buf_mmap.go b/src/cmd/internal/bio/buf_mmap.go index e428952bc9..3543637154 100644 --- a/src/cmd/internal/bio/buf_mmap.go +++ b/src/cmd/internal/bio/buf_mmap.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build darwin || dragonfly || freebsd || linux || netbsd || openbsd || (solaris && go1.20) +// +build darwin dragonfly freebsd linux netbsd openbsd solaris,go1.20 package bio diff --git a/src/cmd/internal/bio/buf_nommap.go b/src/cmd/internal/bio/buf_nommap.go index e904860983..dab3435260 100644 --- a/src/cmd/internal/bio/buf_nommap.go +++ b/src/cmd/internal/bio/buf_nommap.go @@ -3,6 +3,13 @@ // license that can be found in the LICENSE file. //go:build !darwin && !dragonfly && !freebsd && !linux && !netbsd && !openbsd && !(solaris && go1.20) +// +build !darwin +// +build !dragonfly +// +build !freebsd +// +build !linux +// +build !netbsd +// +build !openbsd +// +build !solaris !go1.20 package bio diff --git a/src/cmd/link/elf_test.go b/src/cmd/link/elf_test.go index 6a94b264a1..5037f5b6c0 100644 --- a/src/cmd/link/elf_test.go +++ b/src/cmd/link/elf_test.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build dragonfly || freebsd || linux || netbsd || openbsd +// +build dragonfly freebsd linux netbsd openbsd package main diff --git a/src/cmd/link/internal/ld/outbuf_mmap.go b/src/cmd/link/internal/ld/outbuf_mmap.go index 7bb728a1f8..d8f8a10db1 100644 --- a/src/cmd/link/internal/ld/outbuf_mmap.go +++ b/src/cmd/link/internal/ld/outbuf_mmap.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build aix || darwin || dragonfly || freebsd || linux || netbsd || openbsd || (solaris && go1.20) +// +build aix darwin dragonfly freebsd linux netbsd openbsd solaris,go1.20 package ld diff --git a/src/cmd/link/internal/ld/outbuf_nofallocate.go b/src/cmd/link/internal/ld/outbuf_nofallocate.go index dd5afc61db..6564bd54a3 100644 --- a/src/cmd/link/internal/ld/outbuf_nofallocate.go +++ b/src/cmd/link/internal/ld/outbuf_nofallocate.go @@ -3,6 +3,7 @@ // license that can be found in the LICENSE file. //go:build !darwin && !linux +// +build !darwin,!linux package ld diff --git a/src/cmd/link/internal/ld/outbuf_nommap.go b/src/cmd/link/internal/ld/outbuf_nommap.go index b1d3d2724f..4154981b09 100644 --- a/src/cmd/link/internal/ld/outbuf_nommap.go +++ b/src/cmd/link/internal/ld/outbuf_nommap.go @@ -3,6 +3,15 @@ // license that can be found in the LICENSE file. //go:build !aix && !darwin && !dragonfly && !freebsd && !linux && !netbsd && !openbsd && !(solaris && go1.20) && !windows +// +build !aix +// +build !darwin +// +build !dragonfly +// +build !freebsd +// +build !linux +// +build !netbsd +// +build !openbsd +// +build !solaris !go1.20 +// +build !windows package ld