diff options
| author | Martin Möhrmann <moehrmann@google.com> | 2017-08-16 23:52:16 +0200 |
|---|---|---|
| committer | Martin Möhrmann <moehrmann@google.com> | 2017-08-17 20:10:23 +0000 |
| commit | b6296426a065b85d4c1abbc7012dd633c33bb3c2 (patch) | |
| tree | 83133ccc77260805be73d703753ca2c37bfd30d9 /src | |
| parent | 541f8fef30adf62a2032b6e275e95507b00feb52 (diff) | |
| download | go-b6296426a065b85d4c1abbc7012dd633c33bb3c2.tar.xz | |
runtime: avoid zeroing hmap fields in makemap twice
Stack allocated hmap structs are explicitly zeroed before being
passed by pointer to makemap.
Heap allocated hmap structs are created with newobject
which also zeroes on allocation.
Therefore, setting the hmap fields to 0 or nil is redundant
since they will have been zeroed when hmap was allocated.
Change-Id: I5fc55b75e9dc5ba69f5e3588d6c746f53b45ba66
Reviewed-on: https://go-review.googlesource.com/56291
Reviewed-by: Keith Randall <khr@golang.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/runtime/hashmap.go | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/src/runtime/hashmap.go b/src/runtime/hashmap.go index 3e413e52f7..28ea376cf4 100644 --- a/src/runtime/hashmap.go +++ b/src/runtime/hashmap.go @@ -335,15 +335,10 @@ func makemap(t *maptype, hint int64, h *hmap, bucket unsafe.Pointer) *hmap { if h == nil { h = (*hmap)(newobject(t.hmap)) } - h.count = 0 h.B = B h.extra = extra - h.flags = 0 h.hash0 = fastrand() h.buckets = buckets - h.oldbuckets = nil - h.nevacuate = 0 - h.noverflow = 0 return h } |