[go] cmd/link: align covctrs symbol

0 views
Skip to first unread message

Keith Randall (Gerrit)

unread,
Feb 6, 2026, 6:22:43 PM (2 days ago) Feb 6
to Keith Randall, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Keith Randall, Cherry Mui, Go LUCI, Than McIntosh, golang-co...@googlegroups.com

Keith Randall submitted the change

Change information

Commit message:
cmd/link: align covctrs symbol

If we start the covctrs blob at an odd alignment, then covctrs will
not be correctly aligned. Each individual entry is aligned properly,
but the start marker may be before any padding inserted to enforce
that alignment.

Fixes #58936
Change-Id: I802fbe40eacfa5a3c8c4864e078b0e078da956d5
Reviewed-by: Cherry Mui <cher...@google.com>
Reviewed-by: Keith Randall <k...@google.com>
Files:
  • M src/cmd/cover/cover_test.go
  • A src/cmd/cover/testdata/align.go
  • A src/cmd/cover/testdata/align_test.go
  • M src/cmd/link/internal/ld/data.go
Change size: S
Delta: 4 files changed, 37 insertions(+), 0 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Cherry Mui, +1 by Keith Randall
  • 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: I802fbe40eacfa5a3c8c4864e078b0e078da956d5
Gerrit-Change-Number: 733740
Gerrit-PatchSet: 2
Gerrit-Owner: Keith Randall <k...@golang.org>
Gerrit-Reviewer: Cherry Mui <cher...@google.com>
Gerrit-Reviewer: Keith Randall <k...@golang.org>
Gerrit-Reviewer: Keith Randall <k...@google.com>
Gerrit-Reviewer: Than McIntosh <th...@golang.org>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages