From 2de0dcc1614fc0aeb0dfbfba7899fbd3555e567f Mon Sep 17 00:00:00 2001 From: Nick Travers Date: Wed, 7 Jan 2026 08:49:38 -0800 Subject: [PATCH] spanlatch: remove BenchmarkAtomicLoad benchmark This benchmark was a simple wrapper around `atomic.LoadInt32` that was used to demonstrate the performance characteristics of atomic loads compared to channel operations. Since it benchmarks a standard library primitive rather than any CockroachDB code, it is not useful to keep. Touches: #160253 Release note: None --- pkg/kv/kvserver/spanlatch/signal_test.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/pkg/kv/kvserver/spanlatch/signal_test.go b/pkg/kv/kvserver/spanlatch/signal_test.go index 03eb63603f25..459bf19171f7 100644 --- a/pkg/kv/kvserver/spanlatch/signal_test.go +++ b/pkg/kv/kvserver/spanlatch/signal_test.go @@ -7,7 +7,6 @@ package spanlatch import ( "sync" - "sync/atomic" "testing" "github.com/stretchr/testify/require" @@ -177,10 +176,3 @@ func BenchmarkMultiSelectClosedChan(b *testing.B) { } } } - -func BenchmarkAtomicLoad(b *testing.B) { - a := int32(1) - for i := 0; i < b.N; i++ { - _ = atomic.LoadInt32(&a) - } -}