]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: rename map implementation and test files to use a common prefix
authorMartin Möhrmann <moehrmann@google.com>
Sat, 27 Jan 2018 11:48:15 +0000 (12:48 +0100)
committerMartin Möhrmann <moehrmann@google.com>
Sat, 17 Feb 2018 14:57:32 +0000 (14:57 +0000)
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>
src/cmd/compile/internal/gc/reflect.go
src/cmd/compile/internal/gc/walk.go
src/reflect/type.go
src/runtime/map.go [moved from src/runtime/hashmap.go with 100% similarity]
src/runtime/map_benchmark_test.go [moved from src/runtime/mapspeed_test.go with 100% similarity]
src/runtime/map_fast.go [moved from src/runtime/hashmap_fast.go with 100% similarity]
src/runtime/map_test.go

index 66b1a8e1861c9f56b45bb2a7e139c11844bca93c..e556409d4b650a984f7b7ebd5af7e0fd18584176 100644 (file)
@@ -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
@@ -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]
@@ -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
@@ -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]),
@@ -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
@@ -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.
index 51def75a3333d83bf11a94963c2ac7a67c52f3ce..a0b077b245598177a634dc49e34f386f97e1e6c1 100644 (file)
@@ -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 {
@@ -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)
@@ -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
        }
index 171fc07bfe9535e59fee3e8c565d668f30801d67..716ab0c9fb898f194e6f4818b2841a9693cd393a 100644 (file)
@@ -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.
@@ -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 {
similarity index 100%
rename from src/runtime/hashmap.go
rename to src/runtime/map.go
index 6ed655de0ab661d7866399c4876cb8348d60a1bb..b12b09eeb61c514e3fe65ac5e3d1242b1768f63f 100644 (file)
@@ -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{}