[go] cmd/compile: remove HI LO mips support from regalloc

2 views
Skip to first unread message

Gopher Robot (Gerrit)

unread,
Jul 25, 2026, 9:25:06 AM (yesterday) Jul 25
to Jorropo, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, David Chase, Keith Randall, Keith Randall, golang...@luci-project-accounts.iam.gserviceaccount.com, Martin Möhrmann, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:
cmd/compile: remove HI LO mips support from regalloc

This is a plain lie but the alternative would be to
make HI LO incompatible with load store and spilling
but this requires a register allocator for the register allocator,
which would be complex and low ROI.
Change-Id: Idf53dc619bcf40afcdac5142efa7f6f43b63a9df
Auto-Submit: Jorropo <jorro...@gmail.com>
Reviewed-by: Keith Randall <k...@google.com>
Reviewed-by: David Chase <drc...@google.com>
Reviewed-by: Keith Randall <k...@golang.org>
Files:
  • M src/cmd/compile/internal/mips/ssa.go
  • M src/cmd/compile/internal/mips64/ssa.go
  • M src/cmd/compile/internal/ssa/_gen/MIPS64Ops.go
  • M src/cmd/compile/internal/ssa/_gen/MIPSOps.go
  • M src/cmd/compile/internal/ssa/opGen.go
Change size: L
Delta: 5 files changed, 153 insertions(+), 242 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: Idf53dc619bcf40afcdac5142efa7f6f43b63a9df
Gerrit-Change-Number: 805460
Gerrit-PatchSet: 3
Gerrit-Owner: Jorropo <jorro...@gmail.com>
Gerrit-Reviewer: David Chase <drc...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Jorropo <jorro...@gmail.com>
Gerrit-Reviewer: Keith Randall <k...@golang.org>
Gerrit-Reviewer: Keith Randall <k...@google.com>
Gerrit-Reviewer: Martin Möhrmann <moeh...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages