diff options
| author | Ilya Tocar <ilya.tocar@intel.com> | 2018-03-01 16:52:27 -0600 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2018-03-09 19:37:39 +0000 |
| commit | 91102bf723c2e0912cbc4621f03827c0c3062128 (patch) | |
| tree | 95b04f92026b9beff1aa89d089e34fdc2185882e /src/runtime/panic.go | |
| parent | 4902778607938459fe07d22b8445250fd7987917 (diff) | |
| download | go-91102bf723c2e0912cbc4621f03827c0c3062128.tar.xz | |
runtime: use bytes.IndexByte in findnull
bytes.IndexByte is heavily optimized. Use it in findnull.
This is second attempt, similar to CL97523.
In this version we never call IndexByte on region of memory,
that crosses page boundary. A bit slower than CL97523,
but still fast:
name old time/op new time/op delta
GoString-6 164ns ± 2% 118ns ± 0% -28.00% (p=0.000 n=10+6)
findnull is also used in gostringnocopy,
which is used in many hot spots in the runtime.
Fixes #23830
Change-Id: Id843dd4f65a34309d92bdd8df229e484d26b0cb2
Reviewed-on: https://go-review.googlesource.com/98015
Run-TryBot: Ilya Tocar <ilya.tocar@intel.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/runtime/panic.go')
0 files changed, 0 insertions, 0 deletions