From d6d33f678dfe17e58b1d2f4b4b9d1487cd625c12 Mon Sep 17 00:00:00 2001
From: Shahar Kohanim <skohanim@gmail.com>
Date: Sun, 20 Mar 2016 11:52:41 +0200
Subject: [PATCH] cmd/link: use encbuf when writing integers
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

name       old s/op   new s/op   delta
LinkCmdGo  0.59 ± 6%  0.58 ± 5%  -1.61%  (p=0.000 n=99+99)

GOGC=off:
name       old s/op   new s/op   delta
LinkCmdGo  0.50 ± 3%  0.49 ± 3%  -1.28%  (p=0.000 n=98+99)

Change-Id: I737ae056214999441a210c69ec0cf4febc39a715
Reviewed-on: https://go-review.googlesource.com/20914
Reviewed-by: David Crawshaw <crawshaw@golang.org>
Run-TryBot: David Crawshaw <crawshaw@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
---
 src/cmd/link/internal/ld/symtab.go | 48 +++++++-----------------------
 1 file changed, 10 insertions(+), 38 deletions(-)

diff --git a/src/cmd/link/internal/ld/symtab.go b/src/cmd/link/internal/ld/symtab.go
index 60372c988f..6d1175b1c6 100644
--- a/src/cmd/link/internal/ld/symtab.go
+++ b/src/cmd/link/internal/ld/symtab.go
@@ -286,15 +286,13 @@ func Asmplan9sym() {
 
 var symt *LSym
 
-func Wputl(w uint16) {
-	Cput(uint8(w))
-	Cput(uint8(w >> 8))
-}
+func Wputb(w uint16) { Cwrite(Append16b(encbuf[:0], w)) }
+func Lputb(l uint32) { Cwrite(Append32b(encbuf[:0], l)) }
+func Vputb(v uint64) { Cwrite(Append64b(encbuf[:0], v)) }
 
-func Wputb(w uint16) {
-	Cput(uint8(w >> 8))
-	Cput(uint8(w))
-}
+func Wputl(w uint16) { Cwrite(Append16l(encbuf[:0], w)) }
+func Lputl(l uint32) { Cwrite(Append32l(encbuf[:0], l)) }
+func Vputl(v uint64) { Cwrite(Append64l(encbuf[:0], v)) }
 
 func Append16b(b []byte, v uint16) []byte {
 	return append(b, uint8(v>>8), uint8(v))
@@ -303,20 +301,6 @@ func Append16l(b []byte, v uint16) []byte {
 	return append(b, uint8(v), uint8(v>>8))
 }
 
-func Lputb(l uint32) {
-	Cput(uint8(l >> 24))
-	Cput(uint8(l >> 16))
-	Cput(uint8(l >> 8))
-	Cput(uint8(l))
-}
-
-func Lputl(l uint32) {
-	Cput(uint8(l))
-	Cput(uint8(l >> 8))
-	Cput(uint8(l >> 16))
-	Cput(uint8(l >> 24))
-}
-
 func Append32b(b []byte, v uint32) []byte {
 	return append(b, uint8(v>>24), uint8(v>>16), uint8(v>>8), uint8(v))
 }
@@ -324,26 +308,14 @@ func Append32l(b []byte, v uint32) []byte {
 	return append(b, uint8(v), uint8(v>>8), uint8(v>>16), uint8(v>>24))
 }
 
-func Vputb(v uint64) {
-	Lputb(uint32(v >> 32))
-	Lputb(uint32(v))
-}
-
-func Vputl(v uint64) {
-	Lputl(uint32(v))
-	Lputl(uint32(v >> 32))
-}
-
 func Append64b(b []byte, v uint64) []byte {
-	b = Append32b(b, uint32(v>>32))
-	b = Append32b(b, uint32(v))
-	return b
+	return append(b, uint8(v>>56), uint8(v>>48), uint8(v>>40), uint8(v>>32),
+		uint8(v>>24), uint8(v>>16), uint8(v>>8), uint8(v))
 }
 
 func Append64l(b []byte, v uint64) []byte {
-	b = Append32l(b, uint32(v))
-	b = Append32l(b, uint32(v>>32))
-	return b
+	return append(b, uint8(v), uint8(v>>8), uint8(v>>16), uint8(v>>24),
+		uint8(v>>32), uint8(v>>40), uint8(v>>48), uint8(v>>56))
 }
 
 type byPkg []*Library
-- 
2.48.1