diff --git a/src/cmd/go/internal/modload/buildlist.go b/src/cmd/go/internal/modload/buildlist.go index cde4953afa225..aa59611e8193a 100644 --- a/src/cmd/go/internal/modload/buildlist.go +++ b/src/cmd/go/internal/modload/buildlist.go @@ -73,8 +73,8 @@ type Requirements struct { // nested module back into a parent module). direct map[string]bool - graphOnce sync.Once // guards writes to (but not reads from) graph - graph atomic.Value // cachedGraph + graphOnce sync.Once // guards writes to (but not reads from) graph + graph atomic.Pointer[cachedGraph] } // A cachedGraph is a non-nil *ModuleGraph, together with any error discovered @@ -199,7 +199,7 @@ func (rs *Requirements) initVendor(vendorList []module.Version) { mg.g.Require(vendorMod, vendorList) } - rs.graph.Store(cachedGraph{mg, nil}) + rs.graph.Store(&cachedGraph{mg, nil}) }) } @@ -240,9 +240,9 @@ func (rs *Requirements) hasRedundantRoot() bool { func (rs *Requirements) Graph(ctx context.Context) (*ModuleGraph, error) { rs.graphOnce.Do(func() { mg, mgErr := readModGraph(ctx, rs.pruning, rs.rootModules) - rs.graph.Store(cachedGraph{mg, mgErr}) + rs.graph.Store(&cachedGraph{mg, mgErr}) }) - cached := rs.graph.Load().(cachedGraph) + cached := rs.graph.Load() return cached.mg, cached.err } diff --git a/src/cmd/go/internal/modload/load.go b/src/cmd/go/internal/modload/load.go index 69b0c30978da0..060d0cb21a01d 100644 --- a/src/cmd/go/internal/modload/load.go +++ b/src/cmd/go/internal/modload/load.go @@ -1837,7 +1837,7 @@ func (ld *loader) computePatternAll() (all []string) { func (ld *loader) checkMultiplePaths() { mods := ld.requirements.rootModules if cached := ld.requirements.graph.Load(); cached != nil { - if mg := cached.(cachedGraph).mg; mg != nil { + if mg := cached.mg; mg != nil { mods = mg.BuildList() } }