Commit b3972481 authored by Daniel Martí's avatar Daniel Martí

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í <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: default avatarBrad Fitzpatrick <bradfitz@golang.org>
parent 624e197c
...@@ -104,6 +104,7 @@ func TestIntendedInlining(t *testing.T) { ...@@ -104,6 +104,7 @@ func TestIntendedInlining(t *testing.T) {
"(*Buffer).Bytes", "(*Buffer).Bytes",
"(*Buffer).Cap", "(*Buffer).Cap",
"(*Buffer).Len", "(*Buffer).Len",
"(*Buffer).Grow",
"(*Buffer).Next", "(*Buffer).Next",
"(*Buffer).Read", "(*Buffer).Read",
"(*Buffer).ReadByte", "(*Buffer).ReadByte",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment