diff options
| author | Austin Clements <austin@google.com> | 2017-02-09 14:03:49 -0500 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2017-02-14 15:52:54 +0000 |
| commit | d089a6c7187f1ff85277515405ec6c641588a7ff (patch) | |
| tree | 2eb4d16038ec3ade478a83249cc879898efd5b8b /src/runtime/stack_test.go | |
| parent | c5ebcd2c8ac6c5bdf85ec0a346974efd4b0cbe49 (diff) | |
| download | go-d089a6c7187f1ff85277515405ec6c641588a7ff.tar.gz go-d089a6c7187f1ff85277515405ec6c641588a7ff.zip | |
runtime: remove stack barriers
Now that we don't rescan stacks, stack barriers are unnecessary. This
removes all of the code and structures supporting them as well as
tests that were specifically for stack barriers.
Updates #17503.
Change-Id: Ia29221730e0f2bbe7beab4fa757f31a032d9690c
Reviewed-on: https://go-review.googlesource.com/36620
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/stack_test.go')
| -rw-r--r-- | src/runtime/stack_test.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/runtime/stack_test.go b/src/runtime/stack_test.go index a32b68b630..7a758d1897 100644 --- a/src/runtime/stack_test.go +++ b/src/runtime/stack_test.go @@ -315,17 +315,17 @@ func TestPanicFar(t *testing.T) { defer func() { // At this point we created a large stack and unwound // it via recovery. Force a stack walk, which will - // check the consistency of stack barriers. + // check the stack's consistency. Callers(0, pc) }() defer func() { recover() }() useStackAndCall(100, func() { - // Kick off the GC and make it do something nontrivial - // to keep stack barriers installed for a while. + // Kick off the GC and make it do something nontrivial. + // (This used to force stack barriers to stick around.) xtree = makeTree(18) - // Give the GC time to install stack barriers. + // Give the GC time to start scanning stacks. time.Sleep(time.Millisecond) panic(1) }) |