Stowage/go
2
0
Fork 0
mirror of https://github.com/golang/go.git synced 2025-12-08 06:10:04 +00:00
Code Issues Projects Releases Packages Wiki Activity

[dev.simd] cmd/compile: disable intrinsics test for new simd stuff

Browse source 
this test has been unpossible to get working correctly/
as-expected across architectures, experiments, trybots.
There benefit is a fairy-tale (we're going to check at the merge),
and it costs us time to keep it happy, so for now it is disabled.

Change-Id: Iad913d2590deec606d29bedfa100310e6e9a75bc
Reviewed-on: https://go-review.googlesource.com/c/go/+/677556
Reviewed-by: Junyang Shao <shaojunyang@google.com>
Auto-Submit: David Chase <drchase@google.com>
Auto-Submit: Junyang Shao <shaojunyang@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This commit is contained in:
David Chase 2025-05-30 12:45:11 -04:00 • committed by Gopher Robot
parent 7800f3813c
commit 0ff18a9cca
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/cmd/compile/internal/ssagen/intrinsics_test.go
View file

@ -7,7 +7,6 @@ package ssagen
import (
"flag"
"fmt"
"internal/buildcfg"
"slices"
"strings"
"testing"
@ -17,9 +16,8 @@ import (
var updateIntrinsics = flag.Bool("update", false, "Print an updated intrinsics table")
// TODO turn on always. Current setting insures that simd intrinsics do not leak past experiment,
// but also avoids fail+rubber-stamp-update friction while SIMD is under active development.
var simd = flag.Bool("simd", !buildcfg.Experiment.SIMD, "Also check SIMD intrinsics; default to GOEXPERIMENT = NO simd")
// TODO turn on after SIMD is stable. The time burned keeping this test happy during SIMD development has already well exceeded any plausible benefit.
var simd = flag.Bool("simd", false, "Also check SIMD intrinsics; for now, it is noisy and not helpful")
type testIntrinsicKey struct {
archName string