cmd/compile: arch.MAXWIDTH on amd64 out of date? #38734
Labels
compiler/runtime
Issues related to the Go compiler and/or runtime.
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Comments
mdempsky
added
the
NeedsInvestigation
gopherbot
added
the
compiler/runtime
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
go/src/cmd/compile/internal/amd64/galign.go
Line 17 in b1b6784
Most amd64 implementations only support 48 bits of linear address space, but apparently Intel's Ice Lake product line launched last year and support up to 57 bits.
Theoretically fixing this should be as simple as bumping that constant up, but I'm nervous there are other constants that will need to be bumped too. E.g., probably code that uses the
(*[big]T)(ptr)[:n:n]idiom might need to be tweaked.Noticed in discussion on #37805.
/cc @ianlancetaylor
The text was updated successfully, but these errors were encountered: