]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: Reduce testing for fastlog2 implementation
authorRaul Silvera <rsilvera@google.com>
Wed, 14 Oct 2015 02:49:40 +0000 (19:49 -0700)
committerMinux Ma <minux@golang.org>
Wed, 14 Oct 2015 04:54:33 +0000 (04:54 +0000)
The current fastlog2 testing checks all 64M values in the domain of
interest, which is too much for platforms with no native floating point.

Reduce testing under testing.Short() to speed up builds for those platforms.

Related to #12620

Change-Id: Ie5dcd408724ba91c3b3fcf9ba0dddedb34706cd1
Reviewed-on: https://go-review.googlesource.com/15830
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Joel Sing <jsing@google.com>
Reviewed-by: Minux Ma <minux@golang.org>
Run-TryBot: Minux Ma <minux@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/runtime/fastlog2_test.go

index 8937365d512f2677fd634e2015074cb02f8ef5b5..14f6c908370138bfecc693f834a427a1d4d4a8d0 100644 (file)
@@ -15,7 +15,13 @@ func TestFastLog2(t *testing.T) {
        // implementation over the range of interest for heap sampling.
        const randomBitCount = 26
        var e float64
-       for i := 1; i < 1<<randomBitCount; i++ {
+
+       inc := 1
+       if testing.Short() {
+               // Check 1K total values, down from 64M.
+               inc = 1 << 16
+       }
+       for i := 1; i < 1<<randomBitCount; i += inc {
                l, fl := math.Log2(float64(i)), runtime.Fastlog2(float64(i))
                d := l - fl
                e += d * d