]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: check GOFLAGS not GCFLAGS
authorMichael Pratt <mpratt@google.com>
Fri, 30 Jun 2023 18:32:39 +0000 (14:32 -0400)
committerGopher Robot <gobot@golang.org>
Fri, 30 Jun 2023 22:02:00 +0000 (22:02 +0000)
GCFLAGS doesn't have any defined meaning. cmd/dist enables
mayMoreStackPreempt with GOFLAGS.

For #55160.

Change-Id: I7ac71e4a1a983a56bd228ab5d24294db5cc595f7
Reviewed-on: https://go-review.googlesource.com/c/go/+/507359
Run-TryBot: Michael Pratt <mpratt@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Michael Knyszek <mknyszek@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>

src/runtime/crash_unix_test.go

index 07060b8fabaf8086295735fe3d42e8c075caaf00..6bca2ac66e480a9e5df4f146090e25031e9905a1 100644 (file)
@@ -76,7 +76,7 @@ func TestCrashDumpsAllThreads(t *testing.T) {
 
        testenv.MustHaveGoBuild(t)
 
-       if strings.Contains(os.Getenv("GCFLAGS"), "mayMoreStackPreempt") {
+       if strings.Contains(os.Getenv("GOFLAGS"), "mayMoreStackPreempt") {
                // This test occasionally times out in this debug mode. This is probably
                // revealing a real bug in the scheduler, but since it seems to only
                // affect this test and this is itself a test of a debug mode, it's not