]> Cypherpunks.ru repositories - gostls13.git/commit
cmd/compile,cmd/link,runtime: add start line numbers to func metadata
authorMichael Pratt <mpratt@google.com>
Wed, 7 Sep 2022 17:23:19 +0000 (13:23 -0400)
committerGopher Robot <gobot@golang.org>
Fri, 14 Oct 2022 14:47:12 +0000 (14:47 +0000)
commitf2656f20ea420ada5f15ef06ddf18d2797e18841
treeca71ed64eec8c6027dc5f88af83c4509076adb05
parenta4b4717f23334547f40f90f1457f3dc086259fa3
cmd/compile,cmd/link,runtime: add start line numbers to func metadata

This adds the function "start line number" to runtime._func and
runtime.inlinedCall objects. The "start line number" is the line number
of the func keyword or TEXT directive for assembly.

Subtracting the start line number from PC line number provides the
relative line offset of a PC from the the start of the function. This
helps with source stability by allowing code above the function to move
without invalidating samples within the function.

Encoding start line rather than relative lines directly is convenient
because the pprof format already contains a start line field.

This CL uses a straightforward encoding of explictly including a start
line field in every _func and inlinedCall. It is possible that we could
compress this further in the future. e.g., functions with a prologue
usually have <line of PC 0> == <start line>. In runtime.test, 95% of
functions have <line of PC 0> == <start line>.

According to bent, this is geomean +0.83% binary size vs master and
-0.31% binary size vs 1.19.

Note that //line directives can change the file and line numbers
arbitrarily. The encoded start line is as adjusted by //line directives.
Since this can change in the middle of a function, `line - start line`
offset calculations may not be meaningful if //line directives are in
use.

For #55022.

Change-Id: Iaabbc6dd4f85ffdda294266ef982ae838cc692f6
Reviewed-on: https://go-review.googlesource.com/c/go/+/429638
Run-TryBot: Michael Pratt <mpratt@google.com>
Auto-Submit: Michael Pratt <mpratt@google.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Cherry Mui <cherryyz@google.com>
17 files changed:
src/cmd/asm/internal/asm/asm.go
src/cmd/compile/internal/ir/abi.go
src/cmd/internal/goobj/funcinfo.go
src/cmd/internal/goobj/objfile.go
src/cmd/internal/obj/link.go
src/cmd/internal/obj/objfile.go
src/cmd/internal/obj/plist.go
src/cmd/link/internal/ld/pcln.go
src/cmd/link/internal/loader/loader.go
src/runtime/export_test.go
src/runtime/internal/startlinetest/func_amd64.go [new file with mode: 0644]
src/runtime/internal/startlinetest/func_amd64.s [new file with mode: 0644]
src/runtime/runtime2.go
src/runtime/start_line_amd64_test.go [new file with mode: 0644]
src/runtime/start_line_test.go [new file with mode: 0644]
src/runtime/symtab.go
src/runtime/traceback.go