[go] runtime: remove the unused scanIdx from mspan

0 views

Skip to first unread message

Youlin Feng (Gerrit)

unread,

1:41 AM (17 hours ago) 1:41 AM

to goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Youlin Feng has uploaded the change for review

Commit message

runtime: remove the unused scanIdx from mspan

After CL 700496, mspan.scanIdx is never used, this CL just remove it.

Change-Id: I41ce9902957c0cfa6fbf26b66a2a7787b179376a

Change diff

diff --git a/src/runtime/mheap.go b/src/runtime/mheap.go
index 68dfca4..9b62ee4 100644
--- a/src/runtime/mheap.go
+++ b/src/runtime/mheap.go
@@ -457,12 +457,6 @@
 	// mallocgc, and issue 54596).
 	freeIndexForScan uint16
 
-	// Temporary storage for the object index that caused this span to
-	// be queued for scanning.
-	//
-	// Used only with goexperiment.GreenTeaGC.
-	scanIdx uint16
-
 	// Cache of the allocBits at freeindex. allocCache is shifted
 	// such that the lowest bit corresponds to the bit freeindex.
 	// allocCache holds the complement of allocBits, thus allowing

Change information

Files:

M src/runtime/mheap.go

Change size: XS

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

Open in Gerrit

Related details

Attention set is empty

Submit Requirements:

Code-Review
No-Unresolved-Comments
Review-Enforcement
TryBots-Pass

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

Gerrit

unsatisfied_requirement

satisfied_requirement

open

diffy

Youlin Feng (Gerrit)

unread,

1:42 AM (17 hours ago) 1:42 AM

to goph...@pubsubhelper.golang.org, golang-co...@googlegroups.com

Youlin Feng voted Commit-Queue+1

Commit-Queue