]> Cypherpunks repositories - gostls13.git/commitdiff
runtime: switch AMD64's publicationBarrier to ABIInternal
authorJakub Ciolek <jakub@ciolek.dev>
Wed, 18 Jan 2023 09:54:17 +0000 (10:54 +0100)
committerGopher Robot <gobot@golang.org>
Tue, 24 Jan 2023 19:11:33 +0000 (19:11 +0000)
Shrinks a few runtime functions:

runtime
runtime.mallocgc 2127 -> 2117  (-0.47%)
runtime.(*mheap).initSpan 652 -> 639  (-1.99%)
runtime.(*mspan).userArenaNextFree 825 -> 808  (-2.06%)

file      before   after    Δ       %
runtime.s 512473   512414   -59     -0.012%
total     20843331 20843272 -59     -0.000%

Change-Id: Ifaca7abb6db3467b73945fac71496dd7527e6863
Reviewed-on: https://go-review.googlesource.com/c/go/+/462296
Auto-Submit: Keith Randall <khr@golang.org>
Run-TryBot: Jakub Ciolek <jakub@ciolek.dev>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Keith Randall <khr@golang.org>
src/runtime/asm_amd64.s

index 5d163da132c0a05bb936fa2fd8f6dd88ac33c825..0e72b540be12785c09612e68aed3af45c7fb358b 100644 (file)
@@ -762,7 +762,7 @@ again:
        RET
 
 
-TEXT ·publicationBarrier(SB),NOSPLIT,$0-0
+TEXT ·publicationBarrier<ABIInternal>(SB),NOSPLIT,$0-0
        // Stores are already ordered on x86, so this is just a
        // compile barrier.
        RET