diff options
| author | Keith Randall <khr@golang.org> | 2024-11-14 16:58:07 -0800 |
|---|---|---|
| committer | Keith Randall <khr@golang.org> | 2024-11-17 20:17:05 +0000 |
| commit | 63f762bcdea96889d8ffa406804665b84bda63ab (patch) | |
| tree | 6ec8508a506b512742af1d3415116f90d67ad7be /src/internal/runtime/maps/map.go | |
| parent | 04807d3acf160b270fbec42b7b672d531dec06b7 (diff) | |
| download | go-63f762bcdea96889d8ffa406804665b84bda63ab.tar.xz | |
internal/runtime/maps: eliminate a load from the hot path
typ.Group.Size involves two loads.
Instead cache GroupSize as a separate fields of the map type
so we can get to it in just one load.
Change-Id: I10ffdce1c7f75dcf448da14040fda78f0d75fd1d
Reviewed-on: https://go-review.googlesource.com/c/go/+/627716
Reviewed-by: Cherry Mui <cherryyz@google.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/internal/runtime/maps/map.go')
| -rw-r--r-- | src/internal/runtime/maps/map.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/internal/runtime/maps/map.go b/src/internal/runtime/maps/map.go index 86977bbc2d..969da13432 100644 --- a/src/internal/runtime/maps/map.go +++ b/src/internal/runtime/maps/map.go @@ -296,7 +296,7 @@ func NewMap(mt *abi.SwissMapType, hint uintptr, m *Map, maxAlloc uintptr) *Map { if overflow { return m // return an empty map. } else { - mem, overflow := math.MulUintptr(groups, mt.Group.Size_) + mem, overflow := math.MulUintptr(groups, mt.GroupSize) if overflow || mem > maxAlloc { return m // return an empty map. } |