Last active
October 23, 2024 08:01
-
-
Save smyrman/f7fd1734f9ea20d4648ed359bbcc6ac7 to your computer and use it in GitHub Desktop.
Go copy vs append
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
package benchcp_test | |
import ( | |
"fmt" | |
"testing" | |
) | |
import ( | |
"fmt" | |
"testing" | |
) | |
func BenchmarkAppendAllocInline(b *testing.B) { | |
for _, size := range []int{100, 1000, 10000} { | |
size := size | |
existing := make([]int64, size, size) | |
b.Run(fmt.Sprintf("size=%d", size), func(b *testing.B) { | |
b.ResetTimer() | |
for i := 0; i < b.N; i++ { | |
target := make([]int64, 0, size) | |
target = append(target, existing...) | |
} | |
}) | |
} | |
} | |
func BenchmarkCopyInline(b *testing.B) { | |
for _, size := range []int{100, 1000, 10000} { | |
size := size | |
existing := make([]int64, size, size) | |
b.Run(fmt.Sprintf("size=%d", size), func(b *testing.B) { | |
b.ResetTimer() | |
for i := 0; i < b.N; i++ { | |
target := make([]int64, size, size) | |
copy(target, existing) | |
} | |
}) | |
} | |
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
$ go version | |
go version go1.18.3 darwin/amd64 | |
$ go test -bench=. -benchmem | |
goos: darwin | |
goarch: amd64 | |
pkg: play/bench | |
cpu: Intel(R) Core(TM) i7-6567U CPU @ 3.30GHz | |
BenchmarkAppendAllocInline/size=100-4 7442314 146.4 ns/op 896 B/op 1 allocs/op | |
BenchmarkAppendAllocInline/size=1000-4 929802 1397 ns/op 8192 B/op 1 allocs/op | |
BenchmarkAppendAllocInline/size=10000-4 105584 10860 ns/op 81920 B/op 1 allocs/op | |
BenchmarkCopyInline/size=100-4 7351207 150.4 ns/op 896 B/op 1 allocs/op | |
BenchmarkCopyInline/size=1000-4 920683 1296 ns/op 8192 B/op 1 allocs/op | |
BenchmarkCopyInline/size=10000-4 107530 10376 ns/op 81920 B/op 1 allocs/op | |
PASS | |
ok play/bench 9.644s |
For some reason, with my tests using Go 1.23.2, BenchmarkAppendAllocInline
appears to consistently win by a very thin margin with the benchmark code as is here. But also for some reason, if the make([]int64, size, size)
in BenchmarkCopyInline
is changed to make([]int64, size)
, it starts to consistently win, by a slightly larger margin than it was the other way around.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@deitrix, I don't know how you came across this Gist, but thanks for finding this and pointing out the error. This faulty benchmark have actually led me to assume that appends are probably slightly faster, and if you hadn't pointed it out, I would have continued to believe so.
I have updated and re-run the benchmark. There is now, as you say, no conclusive answer on which one is faster.