fix: workaround to avoid high memory consumption

Author: ldez

pkg/goanalysis/runner.go

@@ -15,7 +15,6 @@ import (
 	"go/token"
 	"runtime"
 	"sort"
-	"strings"
 	"sync"
 
 	"golang.org/x/exp/maps"
@@ -59,12 +58,11 @@ type runner struct {
 	passToPkg      map[*analysis.Pass]*packages.Package
 	passToPkgGuard sync.Mutex
 	sw             *timeutils.Stopwatch
-	goVersion      string // TODO(ldez) temporary workaround
 }
 
-func newRunner(prefix string, logger logutils.Log,
-	pkgCache *pkgcache.Cache, loadGuard *load.Guard, loadMode LoadMode,
-	sw *timeutils.Stopwatch, goVersion string) *runner {
+func newRunner(prefix string, logger logutils.Log, pkgCache *pkgcache.Cache, loadGuard *load.Guard,
+	loadMode LoadMode, sw *timeutils.Stopwatch,
+) *runner {
 	return &runner{
 		prefix:    prefix,
 		log:       logger,
@@ -73,7 +71,6 @@ func newRunner(prefix string, logger logutils.Log,
 		loadMode:  loadMode,
 		passToPkg: map[*analysis.Pass]*packages.Package{},
 		sw:        sw,
-		goVersion: goVersion,
 	}
 }
 
@@ -250,7 +247,6 @@ func (r *runner) analyze(pkgs []*packages.Package, analyzers []*analysis.Analyze
 
 		loadingPackages[pkg] = &loadingPackage{
 			pkg:        pkg,
-			goVersion:  "go" + strings.TrimPrefix(r.goVersion, "go"),
 			imports:    imports,
 			isInitial:  initialPkgs[pkg],
 			log:        r.log,

pkg/goanalysis/runner_loadingpackage.go

@@ -9,6 +9,8 @@ import (
 	"go/types"
 	"os"
 	"reflect"
+	"runtime"
+	"strings"
 	"sync"
 	"sync/atomic"
 
@@ -23,7 +25,6 @@ const unsafePkgName = "unsafe"
 
 type loadingPackage struct {
 	pkg         *packages.Package
-	goVersion   string // TODO(ldez) temporary workaround
 	imports     map[string]*loadingPackage
 	isInitial   bool
 	log         logutils.Log
@@ -151,13 +152,15 @@ func (lp *loadingPackage) loadFromSource(loadMode LoadMode) error {
 		}
 		return imp.Types, nil
 	}
+
 	tc := &types.Config{
 		Importer: importerFunc(importer),
 		Error: func(err error) {
 			pkg.Errors = append(pkg.Errors, lp.convertError(err)...)
 		},
-		GoVersion: lp.goVersion, // TODO(ldez) temporary workaround
+		GoVersion: getGoVersion(),
 	}
+
 	_ = types.NewChecker(tc, pkg.Fset, pkg.Types, pkg.TypesInfo).Files(pkg.Syntax)
 	// Don't handle error here: errors are adding by tc.Error function.
 
@@ -499,3 +502,17 @@ func sizeOfReflectValueTreeBytes(rv reflect.Value, visitedPtrs map[uintptr]struc
 		panic("unknown rv of type " + rv.String())
 	}
 }
+
+// TODO(ldez) temporary workaround
+func getGoVersion() string {
+	goVersion := runtime.Version()
+
+	parts := strings.Fields(goVersion)
+
+	if len(parts) == 0 {
+		return goVersion
+	}
+
+	// When using GOEXPERIMENT, the version returned might look something like "go1.23.0 X:boringcrypto".
+	return parts[0]
+}

pkg/goanalysis/runners.go

@@ -36,12 +36,7 @@ func runAnalyzers(cfg runAnalyzersConfig, lintCtx *linter.Context) ([]result.Iss
 	const stagesToPrint = 10
 	defer sw.PrintTopStages(stagesToPrint)
 
-	var goVersion string
-	if lintCtx.Cfg != nil {
-		goVersion = lintCtx.Cfg.Run.Go
-	}
-
-	runner := newRunner(cfg.getName(), log, lintCtx.PkgCache, lintCtx.LoadGuard, cfg.getLoadMode(), sw, goVersion)
+	runner := newRunner(cfg.getName(), log, lintCtx.PkgCache, lintCtx.LoadGuard, cfg.getLoadMode(), sw)
 
 	pkgs := lintCtx.Packages
 	if cfg.useOriginalPackages() {