]> Cypherpunks repositories - gostls13.git/commit
fmt: avoid reflect.Value.Slice to help escape analysis
authorthepudds <thepudds1460@gmail.com>
Wed, 21 Jun 2023 18:35:53 +0000 (14:35 -0400)
committerGopher Robot <gobot@golang.org>
Thu, 7 Sep 2023 01:59:23 +0000 (01:59 +0000)
commit3466e57354974cd2d17ce902df02e0ac58b8ded5
tree9068ccb5dd8636374ce811ccd9fb3e30ae28ebf9
parenta2f28a48415adb8b8736aa1d5c335f99375b2f3a
fmt: avoid reflect.Value.Slice to help escape analysis

This is part of a series of CLs that aim to reduce how often
interface arguments escape for the print functions in fmt.

Prior to this change, one reason arguments escape is because
printValue calls reflect.Value.Slice, which causes its
value argument to escape (though at this CL, that is
shrouded in the fmt escape analysis logs by other
printValue escape reasons).

This CL avoids that usage by calling f.Bytes instead,
which is possible because we know f is a slice of bytes
or an addressable array of bytes.

Arguments still escape for other reasons.

Change-Id: Ic3f064117a364007e1dd3197cef9d641abbf784a
Reviewed-on: https://go-review.googlesource.com/c/go/+/524940
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Run-TryBot: t hepudds <thepudds1460@gmail.com>
src/fmt/print.go