]> Cypherpunks repositories - gostls13.git/commitdiff
math/big: disable TestNewIntAllocs on noopt builder
authorCuong Manh Le <cuong.manhle.vn@gmail.com>
Mon, 8 Aug 2022 18:39:18 +0000 (01:39 +0700)
committerGopher Robot <gobot@golang.org>
Mon, 8 Aug 2022 20:27:53 +0000 (20:27 +0000)
Since when that test requires inlining, which is disabled on noopt
builder.

Updates #29951

Change-Id: I9d7a0a64015a30d3bfb5ad5d806ea0955657fda3
Reviewed-on: https://go-review.googlesource.com/c/go/+/422039
Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Auto-Submit: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Reviewed-by: Than McIntosh <thanm@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
src/math/big/int_test.go

index 4ebb09d2c2affd925abe36d1f751849822ad48e5..961ba0cdf555b21b90f7e5a8183c8df9a001321c 100644 (file)
@@ -8,6 +8,7 @@ import (
        "bytes"
        "encoding/hex"
        "fmt"
+       "internal/testenv"
        "math"
        "math/rand"
        "strconv"
@@ -1905,6 +1906,9 @@ func TestNewIntMinInt64(t *testing.T) {
 }
 
 func TestNewIntAllocs(t *testing.T) {
+       if strings.HasSuffix(testenv.Builder(), "-noopt") {
+               t.Skip("inlining is disabled on noopt builder")
+       }
        for _, n := range []int64{0, 7, -7, 1 << 30, -1 << 30, 1 << 50, -1 << 50} {
                x := NewInt(3)
                got := testing.AllocsPerRun(100, func() {