From: Daniel Martí Date: Sat, 1 Dec 2018 16:28:27 +0000 (+0000) Subject: cmd/compile: add Buffer.Grow to TestIntendedInlining X-Git-Tag: go1.12beta1~196 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=b397248168fcb26400ac6afb88bf6080497a819e;p=gostls13.git cmd/compile: add Buffer.Grow to TestIntendedInlining golang.org/cl/151977 slightly decreased the cost of inlining an extra call from 60 to 57, since it was a safe change that could help in some scenarios. One notable change spotted in that CL is that bytes.Buffer.Grow is now inlinable, meaning that a fixedbugs test needed updating. For consistency, add the test case to TestIntendedInlining too, alongside other commonly used bytes.Buffer methods. Change-Id: I4fb402fc684ef4c543fc65aea343ca1a4d73a189 Reviewed-on: https://go-review.googlesource.com/c/151979 Run-TryBot: Daniel Martí TryBot-Result: Gobot Gobot Reviewed-by: Brad Fitzpatrick --- diff --git a/src/cmd/compile/internal/gc/inl_test.go b/src/cmd/compile/internal/gc/inl_test.go index ba74981e9a..58d13f2dcf 100644 --- a/src/cmd/compile/internal/gc/inl_test.go +++ b/src/cmd/compile/internal/gc/inl_test.go @@ -104,6 +104,7 @@ func TestIntendedInlining(t *testing.T) { "(*Buffer).Bytes", "(*Buffer).Cap", "(*Buffer).Len", + "(*Buffer).Grow", "(*Buffer).Next", "(*Buffer).Read", "(*Buffer).ReadByte",