]> Cypherpunks.ru repositories - gostls13.git/commit
misc/cgo/testasan: drop test
authorAustin Clements <austin@google.com>
Fri, 14 Oct 2022 18:19:14 +0000 (14:19 -0400)
committerGopher Robot <gobot@golang.org>
Fri, 14 Oct 2022 20:17:35 +0000 (20:17 +0000)
commite612d212c6836407b3274942a394b3c0d46f6e08
tree7dfe027bbed51a20693b708d5a86fbe4ecfecdb2
parent4e3d805a015153ca4e3382415c2474e4f954ddee
misc/cgo/testasan: drop test

The testasan test was added back in 2013 (CL 10126044), many years
before Go added ASAN support in 2021 (CL 298611). So, in fact,
testasan does not test Go ASAN support at all, as you might expect
(misc/cgo/testsanitizers does that). It's intended to test whether the
Go memory allocator works in a mixed C/Go binary where the C code is
compiled with ASAN. The test doesn't actually use ASAN in any way; it
just simulates where ASAN of 2013 put its shadow mappings. This made
sense to test at the time because Go was picky about where its heap
landed and ASAN happened to put its mappings exactly where Go wanted
to put its heap. These days, Go is totally flexible about its heap
placement, and I wouldn't be surprised if ASAN also works differently.

Given all of this, this test adds almost no value today. Drop it.

For #37486, since it eliminates a non-go-test from dist.

Change-Id: I0292f8efbdc0e1e39650715604535c445fbaa87f
Reviewed-on: https://go-review.googlesource.com/c/go/+/443067
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Austin Clements <austin@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
misc/cgo/testasan/main.go [deleted file]
src/cmd/dist/test.go