From 0e31de280f8593dcee2bedb120eb929503785abf Mon Sep 17 00:00:00 2001 From: Cherry Zhang Date: Wed, 24 Mar 2021 11:02:46 -0400 Subject: [PATCH] math/big: don't require runtime.(*Frame).Next symbol present I don't know why the test requires runtime.(*Frame).Next symbol present in the binary under test. I assume it is just some sanity check? With CL 268479 runtime.(*Frame).Next can be pruned by the linker. Replace it with runtime.main which should always be present. May fix the longtest builders. Change-Id: Id3104c058b2786057ff58be41b1d35aeac2f3073 Reviewed-on: https://go-review.googlesource.com/c/go/+/304431 Trust: Cherry Zhang Run-TryBot: Cherry Zhang TryBot-Result: Go Bot Reviewed-by: Bryan C. Mills --- src/math/big/link_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/math/big/link_test.go b/src/math/big/link_test.go index 42f9cefca0..6e33aa5e5e 100644 --- a/src/math/big/link_test.go +++ b/src/math/big/link_test.go @@ -42,7 +42,7 @@ func main() {} if err != nil { t.Fatalf("nm: %v, %s", err, nm) } - const want = "runtime.(*Frames).Next" + const want = "runtime.main" if !bytes.Contains(nm, []byte(want)) { // Test the test. t.Errorf("expected symbol %q not found", want) -- 2.50.0