[go] runtime: remove obsolete "explicit frees" comment from heap profiler

0 views

Skip to first unread message

Gopher Robot (Gerrit)

unread,

Apr 24, 2026, 12:42:45 PM (20 hours ago) Apr 24

to Felix Geisendörfer, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Carlos Amedee, Keith Randall, t hepudds, Nick Ripley, Austin Clements, Keith Randall, Michael Knyszek, golang...@luci-project-accounts.iam.gserviceaccount.com, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:

runtime: remove obsolete memory profiler comment

The runtime's internal free function was removed in
https://golang.org/cl/116390043, and the corresponding accounting code
in mProf_Free was cleaned up in
https://go-review.googlesource.com/16593. All frees now go through GC
sweeping and are accounted to cycle C+1 via mProf_Free.

Change-Id: I1923bd249fa1d718356de4c51653d7c96a6a6964

Reviewed-on: https://go-review.googlesource.com/c/go/+/768020

Reviewed-by: Nick Ripley <nick....@datadoghq.com>

Reviewed-by: t hepudds <thepud...@gmail.com>

Auto-Submit: Keith Randall <k...@google.com>

Reviewed-by: Carlos Amedee <car...@golang.org>

Reviewed-by: Keith Randall <k...@google.com>

LUCI-TryBot-Result: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>

Files:

M src/runtime/mprof.go

Change size: XS

Delta: 1 file changed, 0 insertions(+), 1 deletion(-)

Branch: refs/heads/master

Submit Requirements:

Code-Review: +1 by Carlos Amedee, +2 by Nick Ripley, +1 by Keith Randall, +2 by t hepudds
TryBots-Pass: LUCI-TryBot-Result+1 by golang...@luci-project-accounts.iam.gserviceaccount.com

Open in Gerrit

Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings.

Gerrit

open

diffy

satisfied_requirement

Reply all

Reply to author

Forward

0 new messages