Skip to content

Commit 127e590

Browse files
committed
tsan: update Go x86 build rules to back off to sse3
This is a partial revert of https://reviews.llvm.org/D106948, changing just the Go build rules to remove -msse4.2 and revert back to -msse3, so as to preserve support for older x86 machines. More details at golang/go#53743. Reviewed By: dvyukov Differential Revision: https://reviews.llvm.org/D129482
1 parent 1a2bd44 commit 127e590

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

compiler-rt/lib/tsan/go/build.bat

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,5 +59,8 @@ gcc ^
5959
-DSANITIZER_DEBUG=0 ^
6060
-O3 ^
6161
-fomit-frame-pointer ^
62-
-msse4.2 ^
62+
-msse3 ^
6363
-std=c++14
64+
65+
rem "-msse3" used above to ensure continued support of older
66+
rem cpus (for now), see https://github.com/golang/go/issues/53743.

compiler-rt/lib/tsan/go/buildgo.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,9 @@ if [ "`uname -a | grep Linux`" != "" ]; then
5959
ARCHCFLAGS="-m64 -mcpu=power8 -fno-function-sections"
6060
elif [ "`uname -a | grep x86_64`" != "" ]; then
6161
SUFFIX="linux_amd64"
62-
ARCHCFLAGS="-m64 -msse4.2"
62+
# -msse3 used below to ensure continued support of older
63+
# cpus for now, see https://github.com/golang/go/issues/53743.
64+
ARCHCFLAGS="-m64 -msse3"
6365
OSCFLAGS="$OSCFLAGS -ffreestanding -Wno-unused-const-variable -Wno-unknown-warning-option"
6466
elif [ "`uname -a | grep aarch64`" != "" ]; then
6567
SUFFIX="linux_arm64"

0 commit comments

Comments
 (0)