1
0
mirror of https://github.com/golang/go synced 2024-11-24 06:30:22 -07:00

reflect: adjust MapRange allocation test for noopt builder

Change-Id: I55899ff0ed2c3c01f24ab1ccf133ce4236049e39
Reviewed-on: https://go-review.googlesource.com/c/go/+/401074
Run-TryBot: Keith Randall <khr@golang.org>
Reviewed-by: Joseph Tsai <joetsai@digital-static.net>
Reviewed-by: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
This commit is contained in:
Keith Randall 2022-04-19 08:55:04 -07:00
parent 0670afa1b3
commit 689dc17793

View File

@ -11,6 +11,7 @@ import (
"fmt"
"go/token"
"internal/goarch"
"internal/testenv"
"io"
"math"
"math/rand"
@ -373,8 +374,12 @@ func TestMapIterSet(t *testing.T) {
// Calling MapRange should not allocate even though it returns a *MapIter.
// The function is inlineable, so if the local usage does not escape
// the *MapIter, it can remain stack allocated.
if got != 0 {
t.Errorf("wanted 0 alloc, got %d", got)
want := 0
if strings.HasSuffix(testenv.Builder(), "-noopt") {
want = 1 // no inlining with the noopt builder
}
if got != want {
t.Errorf("wanted %d alloc, got %d", want, got)
}
}