From a90f9a00ca34251dcee80eacff241871c43ac18f Mon Sep 17 00:00:00 2001 From: Austin Clements Date: Mon, 4 Dec 2017 10:58:15 -0500 Subject: runtime: consolidate mheap.lookup* and spanOf* I think we'd forgotten about the mheap.lookup APIs when we introduced spanOf*, but, at any rate, the spanOf* functions are used far more widely at this point, so this CL eliminates the mheap.lookup* functions in favor of spanOf*. Change-Id: I15facd0856e238bb75d990e838a092b5bef5bdfc Reviewed-on: https://go-review.googlesource.com/85879 Run-TryBot: Austin Clements TryBot-Result: Gobot Gobot Reviewed-by: Rick Hudson --- src/runtime/stack.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/runtime/stack.go') diff --git a/src/runtime/stack.go b/src/runtime/stack.go index 6149838b6d..9ed6b1d774 100644 --- a/src/runtime/stack.go +++ b/src/runtime/stack.go @@ -209,7 +209,7 @@ func stackpoolalloc(order uint8) gclinkptr { // Adds stack x to the free pool. Must be called with stackpoolmu held. func stackpoolfree(x gclinkptr, order uint8) { - s := mheap_.lookup(unsafe.Pointer(x)) + s := spanOfUnchecked(uintptr(x)) if s.state != _MSpanManual { throw("freeing stack not in a stack span") } @@ -455,7 +455,7 @@ func stackfree(stk stack) { c.stackcache[order].size += n } } else { - s := mheap_.lookup(v) + s := spanOfUnchecked(uintptr(v)) if s.state != _MSpanManual { println(hex(s.base()), v) throw("bad span state") -- cgit v1.3