[go] cmd/compile: repair mask optimizations

0 views
Skip to first unread message

David Chase (Gerrit)

unread,
Jun 26, 2026, 3:46:27 PM (yesterday) Jun 26
to goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Junyang Shao, Cherry Mui, golang...@luci-project-accounts.iam.gserviceaccount.com, Sanjay Menakuru, Keith Randall, Gopher Robot, golang-co...@googlegroups.com

David Chase submitted the change

Change information

Commit message:
cmd/compile: repair mask optimizations

Had sense of the instructions wrong.
XORQ means a Q of mask bits (64),
not mask bits for Qs (2/4/8).

Includes a not-comprehensive-across operations
test that IS comprehensive across width x count
combinations, to slightly validate use of "B"
mask operations (8 bits) for results of 2 and 4
element comparisons.

Fixes #80140.
Change-Id: I7537bb3327ae124d216f24e4fa054510d8c7cf3c
Reviewed-by: Junyang Shao <shaoj...@google.com>
Files:
  • M src/cmd/compile/internal/ssa/_gen/AMD64.rules
  • M src/cmd/compile/internal/ssa/rewriteAMD64.go
  • M src/simd/archsimd/internal/simd_test/simd_amd64_test.go
Change size: L
Delta: 3 files changed, 346 insertions(+), 131 deletions(-)
Branch: refs/heads/master
Submit Requirements:
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I7537bb3327ae124d216f24e4fa054510d8c7cf3c
Gerrit-Change-Number: 794680
Gerrit-PatchSet: 3
Gerrit-Owner: David Chase <drc...@google.com>
Gerrit-Reviewer: Cherry Mui <cher...@google.com>
Gerrit-Reviewer: David Chase <drc...@google.com>
Gerrit-Reviewer: Junyang Shao <shaoj...@google.com>
Gerrit-Reviewer: Keith Randall <k...@golang.org>
Gerrit-CC: Gopher Robot <go...@golang.org>
Gerrit-CC: Sanjay Menakuru <balas...@gmail.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages