]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: simplify zerorange on mips64
authorJulian Zhu <jz531210@gmail.com>
Thu, 4 Sep 2025 13:38:47 +0000 (21:38 +0800)
committerGopher Robot <gobot@golang.org>
Fri, 5 Sep 2025 15:06:50 +0000 (08:06 -0700)
Change-Id: I488b55a21eaaf74373c2789a34bf9b3945ced072
Reviewed-on: https://go-review.googlesource.com/c/go/+/700936
Reviewed-by: Michael Pratt <mpratt@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Keith Randall <khr@golang.org>

src/cmd/compile/internal/mips64/ggen.go

index 5f3f3e64d9cc1a2bc4592d0a398987a4ccecf10b..0740d9abe88f3eec99ffe83eb5c961dd9664021b 100644 (file)
@@ -5,7 +5,6 @@
 package mips64
 
 import (
-       "cmd/compile/internal/ir"
        "cmd/compile/internal/objw"
        "cmd/compile/internal/types"
        "cmd/internal/obj"
@@ -13,37 +12,14 @@ import (
 )
 
 func zerorange(pp *objw.Progs, p *obj.Prog, off, cnt int64, _ *uint32) *obj.Prog {
-       if cnt == 0 {
-               return p
+       if cnt%int64(types.PtrSize) != 0 {
+               panic("zeroed region not aligned")
        }
-       if cnt < int64(4*types.PtrSize) {
-               for i := int64(0); i < cnt; i += int64(types.PtrSize) {
-                       p = pp.Append(p, mips.AMOVV, obj.TYPE_REG, mips.REGZERO, 0, obj.TYPE_MEM, mips.REGSP, 8+off+i)
-               }
-       } else if cnt <= int64(128*types.PtrSize) {
-               p = pp.Append(p, mips.AADDV, obj.TYPE_CONST, 0, 8+off-8, obj.TYPE_REG, mips.REGRT1, 0)
-               p.Reg = mips.REGSP
-               p = pp.Append(p, obj.ADUFFZERO, obj.TYPE_NONE, 0, 0, obj.TYPE_MEM, 0, 0)
-               p.To.Name = obj.NAME_EXTERN
-               p.To.Sym = ir.Syms.Duffzero
-               p.To.Offset = 8 * (128 - cnt/int64(types.PtrSize))
-       } else {
-               //      ADDV    $(8+frame+lo-8), SP, r1
-               //      ADDV    $cnt, r1, r2
-               // loop:
-               //      MOVV    R0, (Widthptr)r1
-               //      ADDV    $Widthptr, r1
-               //      BNE             r1, r2, loop
-               p = pp.Append(p, mips.AADDV, obj.TYPE_CONST, 0, 8+off-8, obj.TYPE_REG, mips.REGRT1, 0)
-               p.Reg = mips.REGSP
-               p = pp.Append(p, mips.AADDV, obj.TYPE_CONST, 0, cnt, obj.TYPE_REG, mips.REGRT2, 0)
-               p.Reg = mips.REGRT1
-               p = pp.Append(p, mips.AMOVV, obj.TYPE_REG, mips.REGZERO, 0, obj.TYPE_MEM, mips.REGRT1, int64(types.PtrSize))
-               p1 := p
-               p = pp.Append(p, mips.AADDV, obj.TYPE_CONST, 0, int64(types.PtrSize), obj.TYPE_REG, mips.REGRT1, 0)
-               p = pp.Append(p, mips.ABNE, obj.TYPE_REG, mips.REGRT1, 0, obj.TYPE_BRANCH, 0, 0)
-               p.Reg = mips.REGRT2
-               p.To.SetTarget(p1)
+
+       for cnt != 0 {
+               p = pp.Append(p, mips.AMOVV, obj.TYPE_REG, mips.REGZERO, 0, obj.TYPE_MEM, mips.REGSP, off)
+               cnt -= int64(types.PtrSize)
+               off += int64(types.PtrSize)
        }
 
        return p