diff options
author | Keith Randall <khr@google.com> | 2019-09-16 16:53:23 -0700 |
---|---|---|
committer | Keith Randall <khr@golang.org> | 2019-09-17 17:17:11 +0000 |
commit | 07ad84009865b15d68cba036610d04c66bd3f5e8 (patch) | |
tree | f7b1b8a94869ac0efc080f3716554212927820a1 /src/runtime/stack_test.go | |
parent | 38543c2813a1075e09693894625421309d8ef333 (diff) | |
download | go-07ad84009865b15d68cba036610d04c66bd3f5e8.tar.gz go-07ad84009865b15d68cba036610d04c66bd3f5e8.zip |
runtime: remove unneeded noinline directives
Now that mid-stack inlining reports backtraces correctly, we no
longer need to protect against inlining in a few critical areas.
Update #19348
Update #28640
Update #34276
Change-Id: Ie68487e6482c3a9509ecf7ecbbd40fe43cee8381
Reviewed-on: https://go-review.googlesource.com/c/go/+/195818
Reviewed-by: David Chase <drchase@google.com>
Diffstat (limited to 'src/runtime/stack_test.go')
-rw-r--r-- | src/runtime/stack_test.go | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/runtime/stack_test.go b/src/runtime/stack_test.go index 143d3a99a0..adfc65384a 100644 --- a/src/runtime/stack_test.go +++ b/src/runtime/stack_test.go @@ -599,9 +599,6 @@ func (s structWithMethod) callers() []uintptr { return pc[:Callers(0, pc)] } -// The noinline prevents this function from being inlined -// into a wrapper. TODO: remove this when issue 28640 is fixed. -//go:noinline func (s structWithMethod) stack() string { buf := make([]byte, 4<<10) return string(buf[:Stack(buf, false)]) |