Skip to content

Commit 4bc3373

Browse files
randall77gopherbot
authored andcommitted
runtime: turn off large memmove tests under asan/msan
Just like we do for race mode. They are just too slow when running with the sanitizers. Fixes #59448 Change-Id: I86e3e3488ec5c4c29e410955e9dc4cbc99d39b84 Reviewed-on: https://go-review.googlesource.com/c/go/+/687535 Reviewed-by: Keith Randall <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]> Reviewed-by: Cherry Mui <[email protected]> Auto-Submit: Keith Randall <[email protected]>
1 parent 88cf0c5 commit 4bc3373

File tree

1 file changed

+6
-4
lines changed

1 file changed

+6
-4
lines changed

src/runtime/memmove_test.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@ import (
88
"crypto/rand"
99
"encoding/binary"
1010
"fmt"
11+
"internal/asan"
12+
"internal/msan"
1113
"internal/race"
1214
"internal/testenv"
1315
. "runtime"
@@ -102,8 +104,8 @@ func TestMemmoveLarge0x180000(t *testing.T) {
102104
}
103105

104106
t.Parallel()
105-
if race.Enabled {
106-
t.Skip("skipping large memmove test under race detector")
107+
if race.Enabled || asan.Enabled || msan.Enabled {
108+
t.Skip("skipping large memmove test under sanitizers")
107109
}
108110
testSize(t, 0x180000)
109111
}
@@ -114,8 +116,8 @@ func TestMemmoveOverlapLarge0x120000(t *testing.T) {
114116
}
115117

116118
t.Parallel()
117-
if race.Enabled {
118-
t.Skip("skipping large memmove test under race detector")
119+
if race.Enabled || asan.Enabled || msan.Enabled {
120+
t.Skip("skipping large memmove test under sanitizers")
119121
}
120122
testOverlap(t, 0x120000)
121123
}

0 commit comments

Comments
 (0)