aboutsummaryrefslogtreecommitdiff
path: root/src/simd/archsimd/cpu.go
diff options
context:
space:
mode:
authorJunyang Shao <shaojunyang@google.com>2026-01-22 17:44:32 +0000
committerJunyang Shao <shaojunyang@google.com>2026-01-28 11:06:13 -0800
commit83b232b0af1bd498d3df099eb68e3b1e40df2527 (patch)
tree3d31013c9c8722c0370f970d6b47a7109b37da06 /src/simd/archsimd/cpu.go
parent6aef900af4eeb4a35d3ff1237a41cf3f63c56c24 (diff)
downloadgo-83b232b0af1bd498d3df099eb68e3b1e40df2527.tar.xz
cmd/compile, simd: capture VAES instructions and fix AVX512VAES feature
The code previously filters out VAES-only instructions, this CL added them back. This CL added the VAES feature check following the Intel xed data: XED_ISA_SET_VAES: vaes.7.0.ecx.9 # avx.1.0.ecx.28 This CL also found out that the old AVX512VAES feature check is not checking the correct bits, it also fixes it: XED_ISA_SET_AVX512_VAES_128: vaes.7.0.ecx.9 aes.1.0.ecx.25 avx512f.7.0.ebx.16 avx512vl.7.0.ebx.31 XED_ISA_SET_AVX512_VAES_256: vaes.7.0.ecx.9 aes.1.0.ecx.25 avx512f.7.0.ebx.16 avx512vl.7.0.ebx.31 XED_ISA_SET_AVX512_VAES_512: vaes.7.0.ecx.9 aes.1.0.ecx.25 avx512f.7.0.ebx.16 It restricts to the most strict common set - includes avx512vl for even 512-bits although it doesn't requires it. Change-Id: I4e2f72b312fd2411589fbc12f9ee5c63c09c2e9a Reviewed-on: https://go-review.googlesource.com/c/go/+/738500 Reviewed-by: Cherry Mui <cherryyz@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Diffstat (limited to 'src/simd/archsimd/cpu.go')
-rw-r--r--src/simd/archsimd/cpu.go10
1 files changed, 10 insertions, 0 deletions
diff --git a/src/simd/archsimd/cpu.go b/src/simd/archsimd/cpu.go
index 8069ee7f26..a25556ae65 100644
--- a/src/simd/archsimd/cpu.go
+++ b/src/simd/archsimd/cpu.go
@@ -158,3 +158,13 @@ func (X86Features) FMA() bool {
func (X86Features) SHA() bool {
return cpu.X86.HasSHA
}
+
+// VAES returns whether the CPU supports the VAES feature.
+//
+// If it returns true, then the CPU also supports AVX.
+//
+// VAES is defined on all GOARCHes, but will only return true on
+// GOARCH amd64.
+func (X86Features) VAES() bool {
+ return cpu.X86.HasVAES
+}