New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
runtime: x86 CPU feature optimizations dropped in Go 1.17 #48406
Comments
@gopherbot , please backport to Go 1.17. |
Backport issue(s) opened: #48407 (for 1.17). Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://golang.org/wiki/MinorReleases. |
This is fixed by CL 350151. |
Because of wrong case of letters, the cpu features flags were not set properly for amd64. Fixes golang#48406
Change https://golang.org/cl/350151 mentions this issue: |
Sorry. Why do we think this made it to go1.17? The CL that caused the regression was submitted August 23 after go1.18 tree opened. I cant see that change in the go1.17 release branch: Line 637 in 21a4e67
|
Sorry, I was mistaken about that. |
Worth adding |
Possibly. We already have |
CL 344329 caused the runtime to no longer detect POPCNT, SSE 4.1, or FMA support on amd64. This affects the performance of
math/bits.OnesCount
,math.FMA
, and most of the rounding functions inmath
.This CL was released as part of Go 1.17, so this may be worth a back-port, or maybe the risk is too high for not enough reward.
/cc @martisch
The text was updated successfully, but these errors were encountered: