Skip to content

Commit c06eef2

Browse files
prattmicgopherbot
authored andcommitted
runtime: skip TestTimePprof on sanitizers
This test asserts there is no external code, but the sanitizer runtimes are external code. Fixes #73783. Cq-Include-Trybots: luci.golang.try:gotip-windows-amd64-race Change-Id: I6a6a636cf93b7950e3ea35e00ec2eaf89911d712 Reviewed-on: https://go-review.googlesource.com/c/go/+/675296 Reviewed-by: Michael Knyszek <[email protected]> Auto-Submit: Michael Pratt <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]>
1 parent 988eb0d commit c06eef2

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/runtime/crash_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -854,6 +854,9 @@ func TestTimePprof(t *testing.T) {
854854
case "aix", "darwin", "illumos", "openbsd", "solaris":
855855
t.Skipf("skipping on %s because nanotime calls libc", runtime.GOOS)
856856
}
857+
if race.Enabled || asan.Enabled || msan.Enabled {
858+
t.Skip("skipping on sanitizers because the sanitizer runtime is external code")
859+
}
857860

858861
// Pass GOTRACEBACK for issue #41120 to try to get more
859862
// information on timeout.

0 commit comments

Comments
 (0)