diff options
author | Alan Donovan <adonovan@google.com> | 2014-02-11 16:49:27 -0500 |
---|---|---|
committer | Alan Donovan <adonovan@google.com> | 2014-02-11 16:49:27 -0500 |
commit | 1f29e74bfac701d8dfa40b6aaa35d20a7685ac46 (patch) | |
tree | 2d29697e0e80274528fd7e18c7529eee4d110380 /go/ssa/builder_test.go | |
parent | fc1d9c55052172ecb0557341eda4f17a80a94d6e (diff) | |
download | golang-x-tools-1f29e74bfac701d8dfa40b6aaa35d20a7685ac46.tar.gz |
go.tools/go/types: remove Type.MethodSet() method.
Method-set caching is now performed externally using a MethodSetCache (if desired), not by the Types themselves.
This a minor deoptimization due to the extra maps, but avoids a situation in which method-sets are computed and frozen prematurely. (See b/7114)
LGTM=gri
R=gri
CC=golang-codereviews
https://golang.org/cl/61430045
Diffstat (limited to 'go/ssa/builder_test.go')
-rw-r--r-- | go/ssa/builder_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/go/ssa/builder_test.go b/go/ssa/builder_test.go index 03d77a04b..82332241a 100644 --- a/go/ssa/builder_test.go +++ b/go/ssa/builder_test.go @@ -104,7 +104,7 @@ func main() { if !isExt { t.Fatalf("unexpected name type in main package: %s", mem) } - mset := types.NewPointer(mem.Type()).MethodSet() + mset := prog.MethodSets.MethodSet(types.NewPointer(mem.Type())) for i, n := 0, mset.Len(); i < n; i++ { m := prog.Method(mset.At(i)) // For external types, only synthetic wrappers have code. |