From b6149224499108d4ea6e562fd9fd7eac35f50a6d Mon Sep 17 00:00:00 2001 From: Tobias Klauser Date: Mon, 15 Aug 2022 11:04:03 +0200 Subject: [PATCH] hash/crc32: remove single-use {castagnoli,ieee}ArchImpl bools These are only used once right after being set in castagnoliInit and ieeeInit, respectively. Change-Id: Ifc5df1fe3040b97b1d252cc6d6d28d8fbc132fa5 Reviewed-on: https://go-review.googlesource.com/c/go/+/423854 TryBot-Result: Gopher Robot Reviewed-by: Ian Lance Taylor Reviewed-by: Bryan Mills Auto-Submit: Tobias Klauser Run-TryBot: Tobias Klauser --- src/hash/crc32/crc32.go | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/hash/crc32/crc32.go b/src/hash/crc32/crc32.go index f330fdb77a..1a24d980cc 100644 --- a/src/hash/crc32/crc32.go +++ b/src/hash/crc32/crc32.go @@ -76,16 +76,14 @@ type Table [256]uint32 // using this polynomial. var castagnoliTable *Table var castagnoliTable8 *slicing8Table -var castagnoliArchImpl bool var updateCastagnoli func(crc uint32, p []byte) uint32 var castagnoliOnce sync.Once var haveCastagnoli uint32 func castagnoliInit() { castagnoliTable = simpleMakeTable(Castagnoli) - castagnoliArchImpl = archAvailableCastagnoli() - if castagnoliArchImpl { + if archAvailableCastagnoli() { archInitCastagnoli() updateCastagnoli = archUpdateCastagnoli } else { @@ -104,14 +102,11 @@ var IEEETable = simpleMakeTable(IEEE) // ieeeTable8 is the slicing8Table for IEEE var ieeeTable8 *slicing8Table -var ieeeArchImpl bool var updateIEEE func(crc uint32, p []byte) uint32 var ieeeOnce sync.Once func ieeeInit() { - ieeeArchImpl = archAvailableIEEE() - - if ieeeArchImpl { + if archAvailableIEEE() { archInitIEEE() updateIEEE = archUpdateIEEE } else { -- 2.50.0