]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: merge bytes inline test with the rest
authorDaniel Martí <mvdan@mvdan.cc>
Sun, 24 Sep 2017 16:55:19 +0000 (17:55 +0100)
committerDaniel Martí <mvdan@mvdan.cc>
Mon, 25 Sep 2017 06:52:31 +0000 (06:52 +0000)
commit6945c67e102da9a37725de590b7e825d880ac848
treea54cf639e67b032c23e18af2449a3e511c8207c4
parent0168fdb5520c0a77e9a27822caf307e739d4142a
cmd/compile: merge bytes inline test with the rest

In golang.org/cl/42813, a test was added in the bytes package to check
if a Buffer method was being inlined, using 'go tool nm'.

Now that we have a compiler test that verifies that certain funcs are
inlineable, merge it there. Knowing whether the funcs are inlineable is
also more reliable than whether or not their symbol appears in the
binary, too. For example, under some circumstances, inlineable funcs
can't be inlined, such as if closures are used.

While at it, add a few more bytes.Buffer methods that are currently
inlined and should clearly stay that way.

Updates #21851.

Change-Id: I62066e32ef5542d37908bd64f90bda51276da4de
Reviewed-on: https://go-review.googlesource.com/65658
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
Reviewed-by: Marvin Stenger <marvin.stenger94@gmail.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/bytes/buffer_test.go
src/cmd/compile/internal/gc/inl_test.go