]> Cypherpunks repositories - gostls13.git/commitdiff
time: optimize big4
authorIlya Tocar <ilya.tocar@intel.com>
Tue, 21 Aug 2018 19:40:01 +0000 (14:40 -0500)
committerIlya Tocar <ilya.tocar@intel.com>
Tue, 21 Aug 2018 20:12:35 +0000 (20:12 +0000)
Use the same load order in big4 as in encoding/binary.BigEndian.
This order is recognized by the compiler and converted into single load.
This isn't in the hot path, but doesn't hurt readability, so lets do this.

Change-Id: Ib1240d0b278e9d667ad419fe91fa52b23d28cfc0
Reviewed-on: https://go-review.googlesource.com/130478
Run-TryBot: Ilya Tocar <ilya.tocar@intel.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/time/zoneinfo_read.go

index 20f84f00671bff0db6a80c248fe2877dd1b7c2bd..29244db29ec67fe697205efd79d06a87ac475d60 100644 (file)
@@ -55,7 +55,7 @@ func (d *dataIO) big4() (n uint32, ok bool) {
                d.error = true
                return 0, false
        }
-       return uint32(p[0])<<24 | uint32(p[1])<<16 | uint32(p[2])<<8 | uint32(p[3]), true
+       return uint32(p[3]) | uint32(p[2])<<8 | uint32(p[1])<<16 | uint32(p[0])<<24, true
 }
 
 func (d *dataIO) byte() (n byte, ok bool) {