[go] runtime, cmd/link: tighten search for stackObjectRecord

0 views
Skip to first unread message

Gopher Robot (Gerrit)

unread,
Dec 12, 2025, 4:23:22 PM (4 days ago) Dec 12
to Ian Lance Taylor, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, David Chase, Cherry Mui, Go LUCI, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:
runtime, cmd/link: tighten search for stackObjectRecord

A stackObjectRecord should always be in funcdata, between gofunc
and the end of pclntab, except for the special case of
methodValueCallFrameObjs, which should always be in noptrbss.
Adjust the two loops that look for the moduledata corresponding
to a stackObjectRecord to search more precisely, rather than
relying on datap.end.

Closely based on a patch by Michael Stapelberg.

For #76038
Change-Id: I751801d8fd030af751825a67905b2a343280e7d9
Reviewed-by: David Chase <drc...@google.com>
Auto-Submit: Ian Lance Taylor <ia...@golang.org>
Reviewed-by: Cherry Mui <cher...@google.com>
Files:
  • M src/cmd/link/internal/ld/symtab.go
  • M src/runtime/stack.go
  • M src/runtime/stkframe.go
  • M src/runtime/symtab.go
Change size: S
Delta: 4 files changed, 10 insertions(+), 2 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by David Chase, +2 by Cherry Mui
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: go
Gerrit-Branch: master
Gerrit-Change-Id: I751801d8fd030af751825a67905b2a343280e7d9
Gerrit-Change-Number: 728840
Gerrit-PatchSet: 2
Gerrit-Owner: Ian Lance Taylor <ia...@golang.org>
Gerrit-Reviewer: Cherry Mui <cher...@google.com>
Gerrit-Reviewer: David Chase <drc...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Ian Lance Taylor <ia...@golang.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages