diff options
author | Michael Pratt <mpratt@google.com> | 2023-06-30 14:32:39 -0400 |
---|---|---|
committer | Gopher Robot <gobot@golang.org> | 2023-06-30 22:02:00 +0000 |
commit | 5b72f45dd17314af39627c2fcac0fbc099b67603 (patch) | |
tree | 8d01a6a3b8d0bf97cff3c0847f5a6a23fa496f1b | |
parent | 18e17e2cb12837ea2c8582ecdb0cc780f49a1aac (diff) | |
download | go-5b72f45dd17314af39627c2fcac0fbc099b67603.tar.gz go-5b72f45dd17314af39627c2fcac0fbc099b67603.zip |
runtime: check GOFLAGS not GCFLAGS
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>
-rw-r--r-- | src/runtime/crash_unix_test.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/crash_unix_test.go b/src/runtime/crash_unix_test.go index 07060b8fab..6bca2ac66e 100644 --- a/src/runtime/crash_unix_test.go +++ b/src/runtime/crash_unix_test.go @@ -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 |