Skip to content

Commit

Permalink
runtime: rename map implementation and test files to use a common prefix
Browse files Browse the repository at this point in the history
Rename all map implementation and test files to use "map"
as a file name prefix instead of "hashmap" for the implementation
and "map" for the test file names.

Change-Id: I7b317c1f7a660b95c6d1f1a185866f2839e69446
Reviewed-on: https://go-review.googlesource.com/90336
Run-TryBot: Martin Möhrmann <moehrmann@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
  • Loading branch information
martisch committed Feb 17, 2018
1 parent 549cb18 commit f4bb25c
Show file tree
Hide file tree
Showing 7 changed files with 12 additions and 12 deletions.
12 changes: 6 additions & 6 deletions src/cmd/compile/internal/gc/reflect.go
Expand Up @@ -72,7 +72,7 @@ func siglt(a, b *Sig) bool {
// the given map type. This type is not visible to users -
// we include only enough information to generate a correct GC
// program for it.
// Make sure this stays in sync with ../../../../runtime/hashmap.go!
// Make sure this stays in sync with ../../../../runtime/map.go!
const (
BUCKETSIZE = 8
MAXKEYSIZE = 128
Expand Down Expand Up @@ -156,7 +156,7 @@ func bmap(t *types.Type) *types.Type {
// buckets can be marked as having no pointers.
// Arrange for the bucket to have no pointers by changing
// the type of the overflow field to uintptr in this case.
// See comment on hmap.overflow in ../../../../runtime/hashmap.go.
// See comment on hmap.overflow in ../../../../runtime/map.go.
otyp := types.NewPtr(bucket)
if !types.Haspointers(valtype) && !types.Haspointers(keytype) {
otyp = types.Types[TUINTPTR]
Expand Down Expand Up @@ -226,7 +226,7 @@ func bmap(t *types.Type) *types.Type {
}

// hmap builds a type representing a Hmap structure for the given map type.
// Make sure this stays in sync with ../../../../runtime/hashmap.go.
// Make sure this stays in sync with ../../../../runtime/map.go.
func hmap(t *types.Type) *types.Type {
if t.MapType().Hmap != nil {
return t.MapType().Hmap
Expand All @@ -246,7 +246,7 @@ func hmap(t *types.Type) *types.Type {
// nevacuate uintptr
// extra unsafe.Pointer // *mapextra
// }
// must match ../../../../runtime/hashmap.go:hmap.
// must match ../../../../runtime/map.go:hmap.
fields := []*types.Field{
makefield("count", types.Types[TINT]),
makefield("flags", types.Types[TUINT8]),
Expand Down Expand Up @@ -276,7 +276,7 @@ func hmap(t *types.Type) *types.Type {
}

// hiter builds a type representing an Hiter structure for the given map type.
// Make sure this stays in sync with ../../../../runtime/hashmap.go.
// Make sure this stays in sync with ../../../../runtime/map.go.
func hiter(t *types.Type) *types.Type {
if t.MapType().Hiter != nil {
return t.MapType().Hiter
Expand All @@ -303,7 +303,7 @@ func hiter(t *types.Type) *types.Type {
// bucket uintptr
// checkBucket uintptr
// }
// must match ../../../../runtime/hashmap.go:hiter.
// must match ../../../../runtime/map.go:hiter.
fields := []*types.Field{
makefield("key", types.NewPtr(t.Key())), // Used in range.go for TMAP.
makefield("val", types.NewPtr(t.Val())), // Used in range.go for TMAP.
Expand Down
6 changes: 3 additions & 3 deletions src/cmd/compile/internal/gc/walk.go
Expand Up @@ -804,7 +804,7 @@ opswitch:
// a = *var
a := n.List.First()

if w := t.Val().Width; w <= 1024 { // 1024 must match ../../../../runtime/hashmap.go:maxZero
if w := t.Val().Width; w <= 1024 { // 1024 must match ../../../../runtime/map.go:maxZero
fn := mapfn(mapaccess2[fast], t)
r = mkcall1(fn, fn.Type.Results(), init, typename(t), r.Left, key)
} else {
Expand Down Expand Up @@ -1178,7 +1178,7 @@ opswitch:
key = nod(OADDR, key, nil)
}

if w := t.Val().Width; w <= 1024 { // 1024 must match ../../../../runtime/hashmap.go:maxZero
if w := t.Val().Width; w <= 1024 { // 1024 must match ../../../../runtime/map.go:maxZero
n = mkcall1(mapfn(mapaccess1[fast], t), types.NewPtr(t.Val()), init, typename(t), map_, key)
} else {
z := zeroaddr(w)
Expand Down Expand Up @@ -2824,7 +2824,7 @@ var mapassign = mkmapnames("mapassign", "ptr")
var mapdelete = mkmapnames("mapdelete", "")

func mapfast(t *types.Type) int {
// Check ../../runtime/hashmap.go:maxValueSize before changing.
// Check ../../runtime/map.go:maxValueSize before changing.
if t.Val().Width > 128 {
return mapslow
}
Expand Down
4 changes: 2 additions & 2 deletions src/reflect/type.go
Expand Up @@ -2154,7 +2154,7 @@ func needKeyUpdate(t *rtype) bool {
}
}

// Make sure these routines stay in sync with ../../runtime/hashmap.go!
// Make sure these routines stay in sync with ../../runtime/map.go!
// These types exist only for GC, so we only fill out GC relevant info.
// Currently, that's just size and the GC program. We also fill in string
// for possible debugging use.
Expand All @@ -2165,7 +2165,7 @@ const (
)

func bucketOf(ktyp, etyp *rtype) *rtype {
// See comment on hmap.overflow in ../runtime/hashmap.go.
// See comment on hmap.overflow in ../runtime/map.go.
var kind uint8
if ktyp.kind&kindNoPointers != 0 && etyp.kind&kindNoPointers != 0 &&
ktyp.size <= maxKeySize && etyp.size <= maxValSize {
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion src/runtime/map_test.go
Expand Up @@ -620,7 +620,7 @@ func TestMapBuckets(t *testing.T) {
// have a nil bucket pointer due to starting with preallocated buckets
// on the stack. Escaping maps start with a non-nil bucket pointer if
// hint size is above bucketCnt and thereby have more than one bucket.
// These tests depend on bucketCnt and loadFactor* in hashmap.go.
// These tests depend on bucketCnt and loadFactor* in map.go.
t.Run("mapliteral", func(t *testing.T) {
for _, tt := range mapBucketTests {
localMap := map[int]int{}
Expand Down

0 comments on commit f4bb25c

Please sign in to comment.