From: LE Manh Cuong <cuong.manhle.vn@gmail.com>
Date: Thu, 2 May 2019 08:48:14 +0000 (+0700)
Subject: cmd/compile: remove outdate TODO in inl.go
X-Git-Tag: go1.13beta1~442
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=19f5c233d1355d98feef37d704eb76ac75886d49;p=gostls13.git

cmd/compile: remove outdate TODO in inl.go

Mid-stack inlining is enable now, see #19348, but we still can not
remove the special case for runtime.heapBits.nextArena, because
runtime.heapBits.next is too complex to be inlined
(cost 96 exceeds budget 80).

Change-Id: I04ea86509074afdc83a3f70d68b8a1a8829763d1
Reviewed-on: https://go-review.googlesource.com/c/go/+/174839
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
---

diff --git a/src/cmd/compile/internal/gc/inl.go b/src/cmd/compile/internal/gc/inl.go
index 5013c55663..addba98e3e 100644
--- a/src/cmd/compile/internal/gc/inl.go
+++ b/src/cmd/compile/internal/gc/inl.go
@@ -343,10 +343,6 @@ func (v *hairyVisitor) visit(n *Node) bool {
 				// runtime.heapBits.next even though
 				// it calls slow-path
 				// runtime.heapBits.nextArena.
-				//
-				// TODO(austin): Once mid-stack
-				// inlining is the default, remove
-				// this special case.
 				break
 			}
 		}