diff options
| author | Matthew Dempsky <mdempsky@google.com> | 2021-04-23 12:18:59 -0700 |
|---|---|---|
| committer | Matthew Dempsky <mdempsky@google.com> | 2021-04-23 20:57:54 +0000 |
| commit | 691e1b84c11d038060807f9c79107996ca2f19ae (patch) | |
| tree | f8f31ea29406493f4cd1640cd6f9271c3b66a318 /test | |
| parent | 768a39975d8851f1c309b163a8eb4b7a5388aa24 (diff) | |
| download | go-691e1b84c11d038060807f9c79107996ca2f19ae.tar.gz go-691e1b84c11d038060807f9c79107996ca2f19ae.zip | |
cmd/compile: generalize fix for lvalue-init evaluation
The previous fix to ensure early evaluation of lvalue-init statements
(CL 312632) added it after we'd already peeled away any array-OINDEX
expressions. But those might have init statements too, so we need to
do this earlier actually and perhaps more than once.
Longer term, lvalue expressions shouldn't have init statements anyway.
But rsc and I both spent a while looking into this earlier in the dev
cycle and couldn't come up with anything reasonable.
Fixes #45706.
Change-Id: I2d19c5ba421b3f019c62eec45774c84cf04b30ef
Reviewed-on: https://go-review.googlesource.com/c/go/+/313011
Trust: Matthew Dempsky <mdempsky@google.com>
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/fixedbugs/issue45706.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/test/fixedbugs/issue45706.go b/test/fixedbugs/issue45706.go index facf488b3d..6518dbf090 100644 --- a/test/fixedbugs/issue45706.go +++ b/test/fixedbugs/issue45706.go @@ -14,3 +14,12 @@ func g() { for i, *(arr[f()]) = range []int{} { } } + +func h() { + var x int + var f func() int + var arr []int + var arr2 [][0]rune + for arr[x], arr2[arr[f()]][x] = range "" { + } +} |