diff options
| author | Michael Anthony Knyszek <mknyszek@google.com> | 2025-07-14 19:23:12 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2025-07-15 12:24:28 -0700 |
| commit | 0c6296ab128eae1980eb81a51b982766a5d7b5fb (patch) | |
| tree | 3a50f4328000e960b3f6e6fd4a9deb3f20a38bf1 /src/runtime/mgcmark_nogreenteagc.go | |
| parent | 397d2117ec5ca7a96324f02dcfc24d8f00b067a0 (diff) | |
| download | go-0c6296ab128eae1980eb81a51b982766a5d7b5fb.tar.xz | |
runtime: have mergeInlineMarkBits also clear the inline mark bits
This is conceptually simpler, as the sweeper doesn't have to worry about
clearing them separately. It also doesn't have a use for them.
This will also be useful to avoiding unnecessary zeroing in
initInlineMarkBits at allocation time. Currently, because it's used in
both span allocation and at sweep time, we cannot blindly trust
needzero.
This change also renames mergeInlineMarkBits to moveInlineMarkBits to
make this change in semantics clearer from the name.
For #73581.
Change-Id: Ib154738a945633b7ff5b2ae27235baa310400139
Reviewed-on: https://go-review.googlesource.com/c/go/+/687936
Auto-Submit: Michael Knyszek <mknyszek@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Michael Pratt <mpratt@google.com>
Diffstat (limited to 'src/runtime/mgcmark_nogreenteagc.go')
| -rw-r--r-- | src/runtime/mgcmark_nogreenteagc.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/mgcmark_nogreenteagc.go b/src/runtime/mgcmark_nogreenteagc.go index c0ca5c21ea..6e4f0c4f72 100644 --- a/src/runtime/mgcmark_nogreenteagc.go +++ b/src/runtime/mgcmark_nogreenteagc.go @@ -24,7 +24,7 @@ func tryDeferToSpanScan(p uintptr, gcw *gcWork) bool { func (s *mspan) initInlineMarkBits() { } -func (s *mspan) mergeInlineMarks(to *gcBits) { +func (s *mspan) moveInlineMarks(to *gcBits) { throw("unimplemented") } |