diff options
author | Michael Anthony Knyszek <mknyszek@google.com> | 2021-04-01 15:02:11 +0000 |
---|---|---|
committer | Michael Knyszek <mknyszek@google.com> | 2021-04-01 22:35:25 +0000 |
commit | 27d306281c54a89ff00e3dad6f4cea790a2f2f2d (patch) | |
tree | 881afd5999f56794089d663bcc783e403fa32ce4 /src/reflect | |
parent | 51cd074c59c770484e7db4788d968c408a278607 (diff) | |
download | go-27d306281c54a89ff00e3dad6f4cea790a2f2f2d.tar.gz go-27d306281c54a89ff00e3dad6f4cea790a2f2f2d.zip |
reflect,runtime: assume register ABI with GOEXPERIMENT=regabiargs
This change causes finalizers, reflect calls, and Windows syscall
callbacks to assume the register ABI when GOEXPERIMENT=regabiargs is
set. That is, when all Go functions are using the new ABI by default,
these features should assume the new ABI too.
For #40724.
Change-Id: Ie4ee66b8085b692e1ff675f01134c9a4703ae1b9
Reviewed-on: https://go-review.googlesource.com/c/go/+/306571
Trust: Michael Knyszek <mknyszek@google.com>
Run-TryBot: Michael Knyszek <mknyszek@google.com>
Reviewed-by: Austin Clements <austin@google.com>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Reviewed-by: Than McIntosh <thanm@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Diffstat (limited to 'src/reflect')
-rw-r--r-- | src/reflect/abi.go | 6 | ||||
-rw-r--r-- | src/reflect/regabiargs_off.go | 10 | ||||
-rw-r--r-- | src/reflect/regabiargs_on.go | 10 |
3 files changed, 23 insertions, 3 deletions
diff --git a/src/reflect/abi.go b/src/reflect/abi.go index 002e4598b9..ab19695edc 100644 --- a/src/reflect/abi.go +++ b/src/reflect/abi.go @@ -28,9 +28,9 @@ import ( // commented out there should be the actual values once // we're ready to use the register ABI everywhere. var ( - intArgRegs = 0 // abi.IntArgRegs - floatArgRegs = 0 // abi.FloatArgRegs - floatRegSize = uintptr(0) // uintptr(abi.EffectiveFloatRegSize) + intArgRegs = abi.IntArgRegs * experimentRegabiArgs + floatArgRegs = abi.FloatArgRegs * experimentRegabiArgs + floatRegSize = uintptr(abi.EffectiveFloatRegSize * experimentRegabiArgs) ) // abiStep represents an ABI "instruction." Each instruction diff --git a/src/reflect/regabiargs_off.go b/src/reflect/regabiargs_off.go new file mode 100644 index 0000000000..655e955474 --- /dev/null +++ b/src/reflect/regabiargs_off.go @@ -0,0 +1,10 @@ +// Copyright 2021 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +//go:build !goexperiment.regabiargs +// +build !goexperiment.regabiargs + +package reflect + +const experimentRegabiArgs = 0 diff --git a/src/reflect/regabiargs_on.go b/src/reflect/regabiargs_on.go new file mode 100644 index 0000000000..0f33b22ff9 --- /dev/null +++ b/src/reflect/regabiargs_on.go @@ -0,0 +1,10 @@ +// Copyright 2021 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +//go:build goexperiment.regabiargs +// +build goexperiment.regabiargs + +package reflect + +const experimentRegabiArgs = 1 |