[go] cmd/go: parse test files in parallel

[Jake Bailey (Gerrit)]

Attention needed from Michael Matloob and Quim Muntal

New activity on the change

Open in Gerrit

Related details

Attention is currently required from:

• Michael Matloob
• Quim Muntal

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

Gerrit-MessageType: comment

Gerrit-Project: go

Gerrit-Branch: master

Gerrit-Change-Id: If5d7efcf46156fac624d542c639db460d5d49c5f

Gerrit-Change-Number: 795742

Gerrit-PatchSet: 1

Gerrit-Owner: Jake Bailey <jacob.b...@gmail.com>

Gerrit-Reviewer: Jake Bailey <jacob.b...@gmail.com>

Gerrit-Reviewer: Michael Matloob <mat...@golang.org>

Gerrit-Reviewer: Quim Muntal <quimm...@gmail.com>

Gerrit-Reviewer: golang...@luci-project-accounts.iam.gserviceaccount.com <golang...@luci-project-accounts.iam.gserviceaccount.com>

Gerrit-Attention: Quim Muntal <quimm...@gmail.com>

Gerrit-Attention: Michael Matloob <mat...@golang.org>

Gerrit-Comment-Date: Tue, 30 Jun 2026 20:06:48 +0000

Gerrit-HasComments: No

Gerrit-Has-Labels: No

[Jake Bailey (Gerrit)]

Attention needed from Michael Matloob and Quim Muntal

Jake Bailey added 1 comment

File src/cmd/go/internal/load/test.go

Line 656, Patchset 1 (Latest): t.Tests = append(t.Tests, r.funcs.Tests...)

Jake Bailey . resolved

I will note that an even better thing on top of this stack is to store this info in modindex like other info; that drops the load time in my example from 1s to 0.2s. But it's a bigger change for sure and does have perf implications for that cache.

Gerrit-Comment-Date: Tue, 30 Jun 2026 20:27:54 +0000

Gerrit-HasComments: Yes

Gerrit-Has-Labels: No