diff options
| author | Keith Randall <khr@golang.org> | 2016-02-22 13:20:38 -0800 |
|---|---|---|
| committer | Keith Randall <khr@golang.org> | 2016-02-23 00:15:38 +0000 |
| commit | bd70bd9cb2f458b23222083a3a11190f080af7fd (patch) | |
| tree | 77c35258aea9e50c677841ef02af3b6a63871c10 /src/runtime/asm_arm.s | |
| parent | 1e00cc1647916b705682f8721b8a9e9e095bf4ee (diff) | |
| download | go-bd70bd9cb2f458b23222083a3a11190f080af7fd.tar.xz | |
runtime: unify memeq and memequal
They do the same thing, except memequal also has the short-circuit
check if the two pointers are equal.
A) We might as well always do the short-circuit check, it is only 2 instructions.
B) The extra function call (memequal->memeq) is expensive.
benchmark old ns/op new ns/op delta
BenchmarkArrayEqual-8 8.56 5.31 -37.97%
No noticeable affect on the former memeq user (maps).
Fixes #14302
Change-Id: I85d1ada59ed11e64dd6c54667f79d32cc5f81948
Reviewed-on: https://go-review.googlesource.com/19843
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/asm_arm.s')
| -rw-r--r-- | src/runtime/asm_arm.s | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/runtime/asm_arm.s b/src/runtime/asm_arm.s index 07894a3a72..5d0206d1c9 100644 --- a/src/runtime/asm_arm.s +++ b/src/runtime/asm_arm.s @@ -750,13 +750,16 @@ TEXT runtime·memhash_varlen(SB),NOSPLIT,$16-12 MOVW R0, ret+8(FP) RET -TEXT runtime·memeq(SB),NOSPLIT,$-4-13 +// memequal(p, q unsafe.Pointer, size uintptr) bool +TEXT runtime·memequal(SB),NOSPLIT,$-4-13 MOVW a+0(FP), R1 MOVW b+4(FP), R2 MOVW size+8(FP), R3 ADD R1, R3, R6 MOVW $1, R0 MOVB R0, ret+12(FP) + CMP R1, R2 + RET.EQ loop: CMP R1, R6 RET.EQ @@ -779,7 +782,7 @@ TEXT runtime·memequal_varlen(SB),NOSPLIT,$16-9 MOVW R0, 4(R13) MOVW R1, 8(R13) MOVW R2, 12(R13) - BL runtime·memeq(SB) + BL runtime·memequal(SB) MOVB 16(R13), R0 MOVB R0, ret+8(FP) RET @@ -866,7 +869,7 @@ loop: MOVB R8, v+16(FP) RET -// TODO: share code with memeq? +// TODO: share code with memequal? TEXT bytes·Equal(SB),NOSPLIT,$0-25 MOVW a_len+4(FP), R1 MOVW b_len+16(FP), R3 |