diff options
author | Cuong Manh Le <cuong.manhle.vn@gmail.com> | 2019-09-05 11:15:59 +0700 |
---|---|---|
committer | Matthew Dempsky <mdempsky@google.com> | 2019-09-06 01:35:46 +0000 |
commit | 1406ece4461cc042c9340efbe331e2d7f6db2f8e (patch) | |
tree | 77b1f371e4ce73c8956f047abc461546e38c3552 /test/escape_goto.go | |
parent | c99598cbd3aa5eab2971ee449b5685f299b89755 (diff) | |
download | go-1406ece4461cc042c9340efbe331e2d7f6db2f8e.tar.gz go-1406ece4461cc042c9340efbe331e2d7f6db2f8e.zip |
cmd/compile: preserve loop depth when evaluating block
Add block method to preserve loop depth when evaluating statements in a
block, so escape analysis can handle looping label more precisely.
Updates #22438
Change-Id: I39b306544a6c0ee3fcbebbe0d0ee735cb71773e6
Reviewed-on: https://go-review.googlesource.com/c/go/+/193517
Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Diffstat (limited to 'test/escape_goto.go')
-rw-r--r-- | test/escape_goto.go | 44 |
1 files changed, 44 insertions, 0 deletions
diff --git a/test/escape_goto.go b/test/escape_goto.go new file mode 100644 index 0000000000..f024a9afe3 --- /dev/null +++ b/test/escape_goto.go @@ -0,0 +1,44 @@ +// errorcheck -0 -m -l + +// Copyright 2019 The Go Authors. All rights reserved. +// Use of this source code is governed by a BSD-style +// license that can be found in the LICENSE file. + +// Test escape analysis for goto statements. + +package escape + +var x bool + +func _() { + var p *int +loop: + if x { + goto loop + } + // BAD: We should be able to recognize that there + // aren't any more "goto loop" after here. + p = new(int) // ERROR "escapes to heap" + _ = p +} + +func _() { + var p *int + if x { + loop: + goto loop + } else { + p = new(int) // ERROR "does not escape" + } + _ = p +} + +func _() { + var p *int + if x { + loop: + goto loop + } + p = new(int) // ERROR "does not escape" + _ = p +} |